-
Notifications
You must be signed in to change notification settings - Fork 62
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
Proposal: Adopt Symfony coding stardards #150
Comments
^ and could probably use even more Symphony components to further reduce the amount of custom code needing to be maintained. @aleksip at a high level, what sort of things would adopting Symphony's coding standards change? Please oh please oh please don't say we need to switch back over to using tabs 😜 |
Love this |
Indeed! Actually, I have some secret plans related to this I'm hoping to advance next week. 😉
Symfony follows PSR-2 which dictates that code MUST use 4 spaces for indenting, not tabs! |
😳🤔 |
Not evil ones, I promise! 😈 😄 Just planning to find out if some custom PL code could be replaced with a drop in Symfony replacement with no side effects! |
Pattern Lab PHP uses many Symfony components, so it might make sense to adopt their coding standards. They also have a nice tool, the PHP Coding Standards Fixer, which we could run on our entire PHP codebase.
The text was updated successfully, but these errors were encountered: