Skip to content
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

Move the metadata cops to ChefModernize #426

Merged
merged 3 commits into from
Dec 9, 2019
Merged

Move the metadata cops to ChefModernize #426

merged 3 commits into from
Dec 9, 2019

Conversation

tas50
Copy link
Contributor

@tas50 tas50 commented Dec 6, 2019

These cops really aren't deprecations. They're just not necessary
anymore. We shouldn't be alerting in chef-analyze that every customer
needs to drop everything and remove long_description from their
metadata. It doesn't warrant that, but if they're already in a cookbook
they should def clean it up.

Signed-off-by: Tim Smith tsmith@chef.io

@tas50 tas50 requested review from a team as code owners December 6, 2019 19:30
These cops really aren't deprecations. They're just not necessary
anymore. We shouldn't be alerting in chef-analyze that every customer
needs to drop everything and remove long_description from their
metadata. It doesn't warrant that, but if they're already in a cookbook
they should def clean it up.

Signed-off-by: Tim Smith <tsmith@chef.io>
Signed-off-by: Tim Smith <tsmith@chef.io>
Signed-off-by: Tim Smith <tsmith@chef.io>
@tas50 tas50 merged commit 1efa1a3 into master Dec 9, 2019
@chef-expeditor chef-expeditor bot deleted the move_metadata_cops branch December 9, 2019 20:17
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.

1 participant