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

Refresh cookbook #38

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Refresh cookbook #38

wants to merge 7 commits into from

Conversation

Annih
Copy link
Contributor

@Annih Annih commented Jul 30, 2024

  1. use Github Actions instead of travis-ci
  2. Bump deps in Gemfile
  3. Refresh specs using latest chefspec, replace Berkshelf by Policyfile
  4. Apply cookstyle
  5. Remove obsolete dependency on windows cookbook

@Annih Annih force-pushed the refresh_cookbook branch from 2abea97 to 4750a6b Compare July 30, 2024 18:52
Annih added 4 commits July 30, 2024 21:12
The only reason we use to have a dependency on Windows cookbook was the
locate_sysnative_cmd helper.
However this helper is not a "which" or "where", it just check whether
the given path exist under the "native" system path.
In our case, powershell.exe is not located there, but deeper in the
directory structure, so "powershell.exe" was returned.
So we don't need this helper and the Windows cookbook :)
@Annih Annih force-pushed the refresh_cookbook branch from 4750a6b to 55ef27c Compare July 30, 2024 19:12
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