-
Notifications
You must be signed in to change notification settings - Fork 83
Update WPCS to dev-develop branch at a specific commit
#163
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
Conversation
|
PHPCS shows |
|
The warning is coming from The only way to resolve this is by not using Aside, a note from the Composer docs about using explicit commit references:
It's definitely a temporary solution in this case, so IMHO it should be fine to temporarily remove |
|
@mukeshpanchal27 The composer install is now failing in CI. |
|
@swissspidy, the CI still shows that warning and throws the error. |
|
The warning is expected to be there still. The step still fails now because of the additional |
|
Thanks, @spacedmonkey, for helping me correct the composer version issue. |
Closes #136
How to test the Change
phpVersion: 8.1in your.wp-env.jsonfile and restart environment.composer updatecommand.Checklist: