Skip to content

Invoke release tasks directly with Bake - #6

Merged
samuel-williams-shopify merged 4 commits into
mainfrom
concise-release-logging
Sep 21, 2026
Merged

samuel-williams-shopify merged 4 commits into
mainfrom
concise-release-logging

Conversation

@samuel-williams-shopify

@samuel-williams-shopify samuel-williams-shopify commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

Release subprocesses logged roughly 13 KB per invocation because the command contained an embedded Ruby program and copied dependency paths.

Invoke bake directly with chdir: set to the worktree. This removes the generated Ruby bootstrap, manual task registry, copied load paths, and environment overrides. The child uses Bake's normal executable and inherits the bundle environment and coverage hooks. Task results continue to use Bake's JSON output.

The concise command uses the existing info-level logging. Add a regression test for the direct invocation and its working directory.

Validation: 41 tests / 95 assertions pass with parallel execution and a fresh coverage run on Ruby 4.0.7, including building the committed version when the parent has an older version loaded, release regeneration, signing, and publication against a local Git remote. RuboCop passes, and the changed executable line is covered. Overall coverage is 90.52%; the existing global 100% coverage gate is unchanged.

@samuel-williams-shopify samuel-williams-shopify changed the title Keep release subprocess logging concise Invoke release tasks directly with Bake Sep 21, 2026
@samuel-williams-shopify
samuel-williams-shopify merged commit 658eafe into main Sep 21, 2026
24 of 28 checks passed
@samuel-williams-shopify
samuel-williams-shopify deleted the concise-release-logging branch September 21, 2026 07:19
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