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

Automated PR: Cookstyle Changes #36

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

kitchen-porter
Copy link
Contributor

Hey!
I ran Cookstyle 7.25.6 against this repo and here are the results.
This repo was selected due to the topics of chef-cookbook

Changes

Issues found and resolved with resources/msys2_package.rb

Issues found and resolved with: resources/msys2_package.rb

 - 1:1 refactor: Chef/Deprecations/ResourceWithoutUnifiedTrue - Set unified_mode true in Chef Infra Client 15.3+ custom resources to ensure they work correctly in Chef Infra Client 18 April 2022 when Unified Mode becomes the default. https://docs.chef.io/workstation/cookstyle/chef_deprecations_resourcewithoutunifiedtrue

Signed-off-by: kitchen-porter <[email protected]>
@kitchen-porter kitchen-porter requested a review from a team as a code owner October 12, 2021 00:07
@damacus damacus added the Release: Major Release to Chef Supermarket as a major change when merged label Feb 10, 2022
@damacus
Copy link
Member

damacus commented Feb 10, 2022

This needs more work.

@damacus damacus added the Waiting on Contributor Awaiting on the person who raised this to update label Feb 10, 2022
@ramereth
Copy link
Contributor

@damacus we need to finish adopting this cookbook

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release: Major Release to Chef Supermarket as a major change when merged Waiting on Contributor Awaiting on the person who raised this to update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants