-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
[WIP] Use ::class keyword when possible #39775
Conversation
fabpot
commented
Jan 10, 2021
Q | A |
---|---|
Branch? | 4.4 |
Bug fix? | no |
New feature? | no |
Deprecations? | no |
Tickets | n/a |
License | MIT |
Doc PR | n/a |
I was thinking about this as well. Can we update fabbot to enforce the usage of the class constant? |
@xabbuh That's probably the job of PHP CS Fixer, and it's not trivial, see PHP-CS-Fixer/PHP-CS-Fixer#2918 for reference |
0e7634d
to
2e223e6
Compare
Merging this up is going to be a huge one-time PITA, but I think we'll have a nicer codebase afterwards. |
I will merge this one up. My strategy with such a PR is always the following: merge everything up, then merge the PR, merge up again and ignore all conflicts. Start again the work on branch n + 1, rinse and repeat. |
This PR was merged into the 4.4 branch. Discussion ---------- [WIP] Use ::class keyword when possible | Q | A | ------------- | --- | Branch? | 4.4 <!-- see below --> | Bug fix? | no | New feature? | no <!-- please update src/**/CHANGELOG.md files --> | Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files --> | Tickets | n/a <!-- prefix each issue number with "Fix #", no need to create an issue if none exist, explain below instead --> | License | MIT | Doc PR | n/a Commits ------- 036a36c Use ::class keyword when possible
This PR was merged into the 4.4 branch. Discussion ---------- [WIP] Use ::class keyword when possible | Q | A | ------------- | --- | Branch? | 4.4 <!-- see below --> | Bug fix? | no | New feature? | no <!-- please update src/**/CHANGELOG.md files --> | Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files --> | Tickets | n/a <!-- prefix each issue number with "Fix #", no need to create an issue if none exist, explain below instead --> | License | MIT | Doc PR | n/a Commits ------- 036a36c Use ::class keyword when possible