-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
[Coding Style] Enable rule PSR12.Files.FileHeader + unify file headers #22132
Conversation
This one actually isn't reviewable at all. GitHub does not even support to show so many changes. |
Thanks for the feedback Stefan. |
@michalkleiner For future PRs of that size it would be better to have smaller chunks. For that one I guess we simply leave it as is. I only had a look through the changes at random. As it's no easily possible to look through such big commits at once without loosing focus 🙈 |
Description:
While applying the CS rule to fix spacing around file headers I noticed there are some inconsistencies, so I've also bulk unified them across the codebase since we were already updating over 2000 files just for the CS rule.
We will add the commit to the
.git-blame-ignore-revs
file as well so it's fitting to update all the headers at once as well.I reviewed all the files manually before committing them using local git UI (Fork) and I can confirm the change is safe and doesn't alter any functionality.
Main changes:
Review