Skip to content

Conversation

@GaryJones
Copy link
Contributor

As per https://getcomposer.org/doc/04-schema.md#keywords by including "static analysis" as a keyword in the composer.json file, Composer 2.4.0-RC1 and later will prompt users if the package is installed with composer require instead of composer require --dev. See composer/composer#10960 for more info.

As per https://getcomposer.org/doc/04-schema.md#keywords by including "static analysis" as a keyword in the `composer.json` file, Composer 2.4.0-RC1 and later will prompt users if the package is installed with `composer require` instead of `composer require --dev`. See composer/composer#10960 for more info.
@Rarst
Copy link
Owner

Rarst commented Sep 4, 2022

No objection, tests say you broke json though. :)

@GaryJones
Copy link
Contributor Author

I opened 30 PRs like these - and yours happened to be towards the end, so it was a silly mistake. Extra comma removed, so you'll probably want to squash these commits on merge.

@Rarst Rarst merged commit f9b257d into Rarst:master Sep 5, 2022
@GaryJones GaryJones deleted the patch-1 branch September 5, 2022 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants