-
-
Notifications
You must be signed in to change notification settings - Fork 33
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
Missing PHPCS dependency #68
Comments
I'm happy to submit a pull request to resolve this issue. But first I'd need someone in charge to tell me what you'd like me to do first. I was thinking of adding Please let me know if there is anything I can do to help. |
Interesting observation. I usually do add Adding It would be possible to add a choice during installation. Personally I'd rather not and let the user decide what coding standard, static code analysis tool etc. he/she would like to install. I would vote for removing the cs-related scripts from |
I would be happy with removing PHP CS entirely and installing it myself with each project. But having it there pre-installed and pre-configured would definitely be convenient for me and possibly newcomers to the framework who might not yet have experience with automated coding standards. So long as it's either removed completely to avoid confusion or working on a fresh install I'm happy. |
Sorry to bother you both but you interacted with me on a PR I submitted recently so I assume you're both working on this repository at the moment. I would like to help resolve this issue. Is there anything I can do to help? As I said in an earlier comment, I'm happy to put together a pull request if you can give me directions on what you'd like to be done. |
Send a patch: easier to review a diff. |
Working on it. |
$ composer check
|
Bug Report
Probably affects other versions but I'm fairly sure that's the version I installed.
Summary
PHPCS comes installed via
laminas/laminas-coding-standard
but that is removed after the installer script is run. As a result thecomposer check
andcomposer cs-check
commands are broken.Current behavior
When you run
composer check
you get the following error message:If you manually install
squizlabs/php_codesniffer
it will find PHPCS but then you get the following error:How to reproduce
Install a fresh instance of mezzio/mezzio-skeleton either using
composer create-project mezzio/mezzio-skeleton
or by manually cloning the repository and runningcomposer install
orcomposer update
.Once installed run the
composer check
orcomposer cs-check
commands.Expected behavior
I would expect to see the output from PHPCS.
The text was updated successfully, but these errors were encountered: