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

Pull in the latest cookstyle and allow dependabot to bump it #2574

Merged
merged 1 commit into from
Feb 7, 2022

Conversation

tas50
Copy link
Contributor

@tas50 tas50 commented Feb 4, 2022

There are lots of cookstyle releases and we want to make sure we can always pull in new releases.

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

@tas50 tas50 requested review from a team as code owners February 4, 2022 17:37
@netlify
Copy link

netlify bot commented Feb 4, 2022

👷 Deploy Preview for chef-supermarket processing.

🔨 Explore the source changes: 3682451

🔍 Inspect the deploy log: https://app.netlify.com/sites/chef-supermarket/deploys/61fd6480b0ec8c0007a0a127

There's lots of cookstyle releases and we want to make sure we can always pull in new releases.

Signed-off-by: Tim Smith <tsmith@chef.io>
@dheerajd-msys
Copy link
Contributor

DCO sign-off is missing @tas50

@tas50
Copy link
Contributor Author

tas50 commented Feb 7, 2022

@dheerajd-msys DCO is there. The bot just isn't the best

@dheerajd-msys
Copy link
Contributor

Adhoc build https://buildkite.com/chef/chef-supermarket-main-omnibus-adhoc/builds/351 is in progress. Can approve post it's success

@@ -82,7 +82,7 @@ group :test do
end

group :development, :test do
gem "chefstyle", "2.1.3"
gem "chefstyle"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why are we using this cookstyle gem in common gems as well as inside group?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is chefstyle which is used for our ruby code vs. cookstyle which is used for cookbook code

@tas50 tas50 merged commit 40fb2f4 into main Feb 7, 2022
@tas50 tas50 deleted the cookstyle_bump branch February 7, 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.

None yet

3 participants