-
Notifications
You must be signed in to change notification settings - Fork 91
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
[shopsys] added BlankLineBeforeStatementFixer for ensuring empty line before statements #2128
Conversation
TomasLudvik
commented
Nov 19, 2020
Q | A |
---|---|
Description, reason for the PR | This PR adds fixer to ensure empty line before: break, continue, declare, do, for, foreach, if, return, switch, throw, try, while and yield |
New feature | Yes |
BC breaks | No |
Fixes issues | ... |
Have you read and signed our License Agreement for contributions? | Yes |
b32b6bb
to
9c309de
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rules should not be in packages/product-feed-heureka/easy-coding-standard.yml
or it's a mistake?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 awesome
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
There is rather emotional discussion, whether this is the best approach in it's current form. After the dust settles, we will continue...
cf251a5
to
b340f52
Compare
…nsuring empty lines before statements applied for: - 'break' - 'continue' - 'declare' - 'do' - 'for' - 'foreach' - 'if' - 'return' - 'switch' - 'throw' - 'try' - 'while' - 'yield'
…e\BlankLineBeforeStatementFixer
b340f52
to
81439ff
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice work. Can you please check the failing build?
Maybe we should make this change blocked by replacing FunctionLengthSniff with one that ignores whitespace. Just a thought
…e with slevomat sniffs - property per class limit sniff is not supported in any other library so it has been removed - slevomat/coding-standars has been updated to newest version - unused private elements sniff has been removed as same check is done by PHPStan
I have replaced it |
…mmenting\InlineDocCommentDeclarationSniff.InvalidFormat"
2fec059
to
c2a512d
Compare