Skip to content

Restrict bolt gem to development group#834

Merged
mhashizume merged 2 commits into
mainfrom
redo_gems
May 12, 2026
Merged

Restrict bolt gem to development group#834
mhashizume merged 2 commits into
mainfrom
redo_gems

Conversation

@joshcooper
Copy link
Copy Markdown
Contributor

@joshcooper joshcooper commented May 8, 2026

The layout of rake tasks and Gemfiles in this repo is tightly coupled with CJC. The latter relies on bundle installing from the top-level sometimes to run beaker-hostgenerator, and sometimes from the acceptance directory when running beaker tests. Rather than fight CJC, revert the previous change and restrict the bolt dependency to the development group and guarded like GEM_BOLT. Make the Gemfile unmanaged until this can be sorted out in the pdk templates.

This resolves the original problem of GH release prep trying to install patron and failing due to lack of libcurl headers.

I was able to run the release prep workflow using the version in this branch: #837

@joshcooper joshcooper changed the title Redo gems Exclude system_tests group when running release prep in GH May 8, 2026
@joshcooper joshcooper marked this pull request as ready for review May 8, 2026 20:34
@joshcooper joshcooper requested review from a team and bastelfreak as code owners May 8, 2026 20:34
@joshcooper joshcooper changed the title Exclude system_tests group when running release prep in GH Restrict bolt gem to development group May 12, 2026
Commit 5a63923 added a hard dependency on bolt,
but it is only needed when running task acceptance tests. Restore the GEM_BOLT
conditional and make the Gemfile unmanaged until this can be sorted out
in the pdk templates.
Comment thread .sync.yml
":system_tests":
- gem: voxpupuli-acceptance
version: '~> 3'

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

These are strictly not needed (since the Gemfile is unmanaged), but I left them for when we re-manage it.

@mhashizume mhashizume merged commit 16d966b into main May 12, 2026
9 checks passed
@mhashizume mhashizume deleted the redo_gems branch May 12, 2026 22:03
@mhashizume mhashizume added the maintenance Maintenance chores are typically excluded from changelogs label May 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Maintenance chores are typically excluded from changelogs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants