Skip to content

feat(ci): add --dry-run flag to bit ci pr#10248

Merged
davidfirst merged 4 commits into
masterfrom
feat/ci-pr-dry-run
Mar 24, 2026
Merged

feat(ci): add --dry-run flag to bit ci pr#10248
davidfirst merged 4 commits into
masterfrom
feat/ci-pr-dry-run

Conversation

@davidfirst
Copy link
Copy Markdown
Member

Adds a --dry-run (-d) flag to bit ci pr that runs the full pipeline (snap + build) but skips exporting to remote. This lets developers debug the CI PR pipeline locally without side effects.

When --dry-run is set, the command performs status verification, import, lane creation, and snap as usual, then logs a summary and returns early — skipping the remote lane deletion, rename, and export steps. The finally cleanup block still runs normally.

Follows the same pattern as --skip-push in bit ci merge.

Copilot AI review requested due to automatic review settings March 24, 2026 16:24
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a --dry-run (-d) option to the bit ci pr command to allow running the PR snap pipeline locally while skipping the remote-finalization steps (remote lane deletion/rename + export), enabling safer local debugging of the CI PR flow.

Changes:

  • Add --dry-run flag to bit ci pr CLI command and plumb it through to CI runtime.
  • Implement early-return behavior in CiMain.snapPrCommit() after snap output when dryRun is set, skipping export/finalization.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
scopes/git/ci/commands/pr.cmd.ts Adds --dry-run option and forwards dryRun into snapPrCommit() execution.
scopes/git/ci/ci.main.runtime.ts Introduces dryRun handling to skip lane finalization/export after a successful snap.

Comment thread scopes/git/ci/commands/pr.cmd.ts
Comment thread scopes/git/ci/ci.main.runtime.ts
Clarify help text that build only runs with --build flag. Keep temp
lane on successful dry-run for debugging, and log how to remove it.
Copilot AI review requested due to automatic review settings March 24, 2026 17:24
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@davidfirst davidfirst enabled auto-merge (squash) March 24, 2026 17:58
@davidfirst davidfirst merged commit 3f4b995 into master Mar 24, 2026
12 checks passed
@davidfirst davidfirst deleted the feat/ci-pr-dry-run branch March 24, 2026 18:22
davidfirst added a commit that referenced this pull request Mar 25, 2026
Adds a `--dry-run` (`-d`) flag to `bit ci pr` that runs the full
pipeline (snap + build) but skips exporting to remote. This lets
developers debug the CI PR pipeline locally without side effects.

When `--dry-run` is set, the command performs status verification,
import, lane creation, and snap as usual, then logs a summary and
returns early — skipping the remote lane deletion, rename, and export
steps. The `finally` cleanup block still runs normally.

Follows the same pattern as `--skip-push` in `bit ci merge`.
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.

3 participants