Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 9, 2025

Commit ee7211a modified the Gemfile to use double quotes and group :chefstyle. This PR standardizes the quote style to single quotes for consistency with Ruby conventions while maintaining group :chefstyle to ensure CI compatibility.

Changes

  • Changed Gemfile to use single quotes consistently throughout
  • Maintained group :chefstyle (not :cookstyle) to match the CI workflow's BUNDLE_WITH: chefstyle expectation
  • Fixes the bundler error: "command not found: chefstyle" in CI

The CI workflow requires the bundler group to be named :chefstyle to properly install and run the cookstyle gem during linting checks.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Reverted Gemfile changes that incorrectly changed quote style. The
original file used single quotes consistently, which matches chefstyle
requirements. Also kept group name as :cookstyle (not :chefstyle) to
match kitchen-vagrant pattern.

Signed-off-by: copilot-swe-agent[bot]

Co-authored-by: damacus <40786+damacus@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix gemspec load path and refactor proxy config method fix: revert Gemfile to use single quotes per chefstyle Nov 9, 2025
Copilot AI requested a review from damacus November 9, 2025 22:07
@tas50 tas50 marked this pull request as ready for review November 9, 2025 23:34
@damacus
Copy link
Contributor

damacus commented Nov 10, 2025

The CI workflow expects `BUNDLE_WITH: chefstyle`, so the Gemfile must
use `group :chefstyle` (not `:cookstyle`). Changed to single quotes
for consistency with Ruby style conventions.

Fixes bundler error: "command not found: chefstyle"

Signed-off-by: copilot-swe-agent[bot]

Co-authored-by: damacus <40786+damacus@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Nov 10, 2025

You're fixing the wrong thing

https://github.com/test-kitchen/kitchen-docker/actions/runs/19215106059/job/54925813681#step:6:1...

Fixed in commit 2663547. The issue was the group name - the CI workflow uses BUNDLE_WITH: chefstyle, so the Gemfile must have group :chefstyle (not :cookstyle). This fixes the "bundler: command not found: chefstyle" error.

Copilot AI changed the title fix: revert Gemfile to use single quotes per chefstyle fix: correct Gemfile to use :chefstyle group for CI compatibility Nov 10, 2025
@damacus damacus closed this Dec 13, 2025
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.

2 participants