-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
feature: PHP8.2 - "readonly" class support #6624
feature: PHP8.2 - "readonly" class support #6624
Conversation
5e9e66c
to
bd0efaf
Compare
ae53bc7
to
2e96eac
Compare
28ede6c
to
980360f
Compare
@SpacePossum we should wait for #6621 to be merged, then rebase this one and change fixture in integration test, so keywords are in wrong order (now only casing is verified, order is preserved). |
There is a utest for the feature to order, so there is no need to wait. |
69abed4
to
358988c
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.
Great work 🙂 I left some comments, hope you'll find them useful.
b170023
to
0e97253
Compare
f971b6b
to
3f4aa21
Compare
3f4aa21
to
fb4a429
Compare
@SpacePossum why did you close this PR? 🤔 |
These are changes initially proposed by SpacePossum in PHP-CS-Fixer#6624 but abandoned due to personal reasons. Since it's accepted by the author to use these changes, I just apply the full patch, so we can continue work.
These are changes initially proposed by SpacePossum in PHP-CS-Fixer#6624 but abandoned due to personal reasons. Since it's accepted by the author to use these changes, I've just applied the full patch, so we can continue work.
These are changes initially proposed by SpacePossum in PHP-CS-Fixer#6624 but abandoned due to personal reasons. Since it's accepted by the author to use these changes, I've just applied the full patch, so we can continue work.
This fixes the current the rules and code so these handle the new
readonly
classes as added in PHP8.2.The
class_definition
rule will now also sort theclass
`modifiers following: https://github.com/php-fig/per-coding-style/blob/9dc210da72285afd5ab10ab0bd34659ff5b0529f/spec.md#46-modifier-keywords