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

Create the local:lint:php command #14

Open
wants to merge 1 commit into
base: 2.x
Choose a base branch
from

Conversation

randallquesadaa
Copy link

Run fire php-lint to see the errors in your code.

@dalin-
Copy link
Contributor

dalin- commented Mar 8, 2024

I don't think this is a good approach. Linting standards should be set in phpcs.xml, not hard-coded for all sites. At most I think we this should be a simple wrapper, so that fire local:lint:php calls a (configurable) command (we could default to ./vendor/bin/phpcs).

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

Successfully merging this pull request may close these issues.

None yet

3 participants