Skip to content
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

Add support for phpcs2 and phpcbf to auto-correct code style #201

Open
grayside opened this issue Nov 9, 2015 · 0 comments
Open

Add support for phpcs2 and phpcbf to auto-correct code style #201

grayside opened this issue Nov 9, 2015 · 0 comments

Comments

@grayside
Copy link
Contributor

grayside commented Nov 9, 2015

This combination of tools could be built into our process to expedite resolving style issues.

  1. Does the code reformatting require review for correctness of style, or even error-free functionality?
  2. Do we want it to run as part of currently read-only operations like grunt validate?
  3. Should we bundle it with other code formatting and cleanup tools, for a top level grunt optimize task? This seems to imply we don't group it with validate or the main build process, and instead provide it to developers to use reactively.
  4. Can we streamline how IDEs might apply formatting correction automatically?

This issue needs someone to set up the tools, run edge case experiments, see what IDE integration with phpstorm, sublime, or atom might look like, and come back with findings so we can make decisions on next steps.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant