-
-
Notifications
You must be signed in to change notification settings - Fork 37
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
README: update with information about Composer >= 2.2 #141
Conversation
As of Composer 2.2.0, Composer plugins need to be explicitly allowed to run. This updates the readme to inform users of this mechanism. Ref: https://blog.packagist.com/composer-2-2/#more-secure-plugin-execution
... to be a little more realistic for current dev.
I was wondering if, based on this, we need to add |
We don't (need to). The thing is, when doing a That's also why in the test scrips in the CI, there a separate call to Other than that, we use |
Reviewed the scripts now and found a few places we could add |
Good work! |
Inspired by a similar README update I saw in the Composer Normalize README, I've added one more commit which shows the CLI command to add the permission. |
Anything I can do to move this PR forward ? |
It was mostly a lack of time that kept this standing still. I had a deadline at work that kept me busy throughout the holidays, and I had to help the kids with school as we were in quarantine. This coming week the kids are back to school and I have time of from work (we made the deadline 🎉), so I plan on catching up here and in other projects. |
README: update with information about Composer >= 2.2
As of Composer 2.2.0, Composer plugins need to be explicitly allowed to run.
This updates the readme to inform users of this mechanism.
Ref: https://blog.packagist.com/composer-2-2/#more-secure-plugin-execution
README: update coding standards dev code sample
... to be a little more realistic for current dev.
🆕 README: add information about adding permission via CLI