-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
[Contributing] [Standards] Do not use spaces inside/around offset accessors #6765
Conversation
…essors | Q | A | ------------- | --- | Doc fix? | yes | New docs? | yes | Applies to | all | Fixed tickets | N/A Ref: PHP-CS-Fixer/PHP-CS-Fixer#2089
which return or throw something. | ||
which return or throw something; | ||
|
||
* Do not use spaces inside/around offset accessors; |
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.
should be full stop
codebase is following the rule (I checked 2.7 branch) |
👍 this is indeed a rule we apply |
I'm thinking how to reword this rule in order to express what we really want, by instance: |
here braces are not offset accessors, they are short array syntax
here, indeed, should be no spaces except space after
again short array syntax so in general, sth like: |
👍 Status: reviewed |
Thanks Javier. |
…und offset accessors (phansys) This PR was squashed before being merged into the 2.7 branch (closes #6765). Discussion ---------- [Contributing] [Standards] Do not use spaces inside/around offset accessors | Q | A | ------------- | --- | Doc fix? | yes | New docs? | yes | Applies to | all | Fixed tickets | N/A Ref: PHP-CS-Fixer/PHP-CS-Fixer#2089 Commits ------- e9fa33d [Contributing] [Standards] Do not use spaces inside/around offset accessors
Ref: PHP-CS-Fixer/PHP-CS-Fixer#2089