Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added wrapping location to allow wrapping before delimiter, fixes #299 #309

Merged
merged 4 commits into from
Feb 3, 2019

Conversation

AlexHaxe
Copy link
Member

@AlexHaxe AlexHaxe commented Feb 3, 2019

fixed anon types and leftCurly=both, fixes #301
fixed same line handling of expressionIf, fixes #304
fixed indentation of object literals, fixes #305
fixed same line handling of cases with object patterns, fixes #306
fixed empty line after class metadata with function, fixes #307
fixed missing space between Comma and POpen, fixes #308
refactored to increase null safety

…xeCheckstyle#299

fixed anon types and leftCurly=both, fixes HaxeCheckstyle#301
fixed same line handling of expressionIf, fixes HaxeCheckstyle#304
fixed indentation of object literals, fixes HaxeCheckstyle#305
fixed same line handling of cases with object patterns, fixes HaxeCheckstyle#306
fixed empty line after class metadata with function, fixes HaxeCheckstyle#307
fixed missing space between Comman and POpen, fixes HaxeCheckstyle#308
refactored to increase null safety
@AlexHaxe AlexHaxe changed the title added wraaping location to allow wrapiping before delimiter, fixes #299 added wrapping location to allow wrapping before delimiter, fixes #299 Feb 3, 2019
@Gama11
Copy link
Member

Gama11 commented Feb 3, 2019

Hm, are you even using code from the safety lib anywhere? I mostly see Null<T> wrapping, but the diff is pretty big. :)

@AlexHaxe
Copy link
Member Author

AlexHaxe commented Feb 3, 2019

I think it's only in two places and haxe 3.x build is still broken, I might drop it for now...

@Gama11
Copy link
Member

Gama11 commented Feb 3, 2019

Would also make it a dependency for vshaxe.

@codecov
Copy link

codecov bot commented Feb 3, 2019

Codecov Report

Merging #309 into master will decrease coverage by 0.06%.
The diff coverage is 91.59%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #309      +/-   ##
==========================================
- Coverage   88.98%   88.92%   -0.07%     
==========================================
  Files          17       17              
  Lines        5084     5226     +142     
  Branches      230      239       +9     
==========================================
+ Hits         4524     4647     +123     
- Misses        330      340      +10     
- Partials      230      239       +9
Impacted Files Coverage Δ
src/formatter/marker/MarkerBase.hx 94.11% <ø> (+2.94%) ⬆️
src/formatter/config/Config.hx 92% <100%> (+7.62%) ⬆️
src/formatter/marker/MarkWhitespace.hx 97.93% <100%> (-0.19%) ⬇️
src/formatter/marker/Indenter.hx 93.64% <100%> (+0.18%) ⬆️
src/formatter/marker/MarkTokenText.hx 99.53% <100%> (ø) ⬆️
src/formatter/codedata/CodeLines.hx 100% <100%> (ø) ⬆️
src/formatter/codedata/CodeLine.hx 92.47% <100%> (ø) ⬆️
src/formatter/marker/MarkLineEnds.hx 87.66% <100%> (ø) ⬆️
src/formatter/marker/wrapping/MarkWrapping.hx 90.21% <100%> (+0.04%) ⬆️
src/formatter/marker/wrapping/MarkWrappingBase.hx 82.23% <81.18%> (-0.54%) ⬇️
... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a050be8...ed0528a. Read the comment docs.

1 similar comment
@codecov
Copy link

codecov bot commented Feb 3, 2019

Codecov Report

Merging #309 into master will decrease coverage by 0.06%.
The diff coverage is 91.59%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #309      +/-   ##
==========================================
- Coverage   88.98%   88.92%   -0.07%     
==========================================
  Files          17       17              
  Lines        5084     5226     +142     
  Branches      230      239       +9     
==========================================
+ Hits         4524     4647     +123     
- Misses        330      340      +10     
- Partials      230      239       +9
Impacted Files Coverage Δ
src/formatter/marker/MarkerBase.hx 94.11% <ø> (+2.94%) ⬆️
src/formatter/config/Config.hx 92% <100%> (+7.62%) ⬆️
src/formatter/marker/MarkWhitespace.hx 97.93% <100%> (-0.19%) ⬇️
src/formatter/marker/Indenter.hx 93.64% <100%> (+0.18%) ⬆️
src/formatter/marker/MarkTokenText.hx 99.53% <100%> (ø) ⬆️
src/formatter/codedata/CodeLines.hx 100% <100%> (ø) ⬆️
src/formatter/codedata/CodeLine.hx 92.47% <100%> (ø) ⬆️
src/formatter/marker/MarkLineEnds.hx 87.66% <100%> (ø) ⬆️
src/formatter/marker/wrapping/MarkWrapping.hx 90.21% <100%> (+0.04%) ⬆️
src/formatter/marker/wrapping/MarkWrappingBase.hx 82.23% <81.18%> (-0.54%) ⬇️
... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a050be8...ed0528a. Read the comment docs.

@codecov
Copy link

codecov bot commented Feb 3, 2019

Codecov Report

Merging #309 into master will decrease coverage by <.01%.
The diff coverage is 91.93%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #309      +/-   ##
==========================================
- Coverage   88.98%   88.98%   -0.01%     
==========================================
  Files          17       17              
  Lines        5084     5229     +145     
  Branches      230      239       +9     
==========================================
+ Hits         4524     4653     +129     
- Misses        330      337       +7     
- Partials      230      239       +9
Impacted Files Coverage Δ
src/formatter/codedata/CodeLine.hx 92.47% <ø> (ø) ⬆️
src/formatter/marker/MarkerBase.hx 94.11% <ø> (+2.94%) ⬆️
src/formatter/config/Config.hx 92% <100%> (+7.62%) ⬆️
src/formatter/marker/Indenter.hx 93.64% <100%> (+0.18%) ⬆️
src/formatter/codedata/CodeLines.hx 100% <100%> (ø) ⬆️
src/formatter/marker/MarkTokenText.hx 99.53% <100%> (ø) ⬆️
src/formatter/marker/MarkLineEnds.hx 87.66% <100%> (ø) ⬆️
src/formatter/marker/wrapping/MarkWrapping.hx 90.21% <100%> (+0.04%) ⬆️
src/formatter/marker/MarkEmptyLines.hx 95.08% <100%> (+0.09%) ⬆️
src/formatter/marker/MarkWhitespace.hx 98.14% <100%> (+0.02%) ⬆️
... and 5 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a050be8...ed0528a. Read the comment docs.

@AlexHaxe
Copy link
Member Author

AlexHaxe commented Feb 3, 2019

Would also make it a dependency for vshaxe.

yeah, maybe not a good idea

@AlexHaxe AlexHaxe merged commit a3fc5a5 into HaxeCheckstyle:master Feb 3, 2019
@AlexHaxe AlexHaxe deleted the fix_some_bugs branch February 3, 2019 23:44
@Gama11
Copy link
Member

Gama11 commented Feb 3, 2019

What's with all those codecov reports? Is that a bug, or is there one for each commit? :D

@AlexHaxe
Copy link
Member Author

AlexHaxe commented Feb 3, 2019

I think they used to post just one and they updated it when new results came in, apparently they just post a fresh one everytime nowadays.
I can look through settings and permissions, but I think if they can post, they can edit their post, so it might be on their end...

@AlexHaxe
Copy link
Member Author

AlexHaxe commented Feb 3, 2019

ok, we had two webhooks for codecov in there, I deleted one, we'll see if that fixes it when the next PR happens...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants