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
DX: Add composer keywords #6781
Conversation
Pull Request Test Coverage Report for Build 4277031245
💛 - Coveralls |
composer.json
Outdated
@@ -2,6 +2,7 @@ | |||
"name": "friendsofphp/php-cs-fixer", | |||
"description": "A tool to automatically fix PHP code style", | |||
"license": "MIT", | |||
"keywords": ["dev", "code style", "code-style", "fixer"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cs, sca, ...
also, are code style
and code-style
really treated differently?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we coud add a lot of stuff here. the only important one is dev
, to make composer work like described above.
we could add anything here. I am open to your suggestions :)
also, are
code style
andcode-style
really treated differently?
I don't know
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd go with fixer
, standards
and static analysis
only, the dev
is not really valuable keyword.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Composer expects one of the following to trigger the desired behavior
dev, testing, static analysis
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
adjusted per feedback
thank you! |
as per composer/composer#10960 this allows composer to warn the user, when installing the tool as a project dependency instead of a dev-dependency