Skip to content

Fix vch land wrong-branch diagnostic#125

Merged
Maples7 merged 1 commit into
masterfrom
fix/124-land-into-diagnostic
May 16, 2026
Merged

Fix vch land wrong-branch diagnostic#125
Maples7 merged 1 commit into
masterfrom
fix/124-land-into-diagnostic

Conversation

@Maples7
Copy link
Copy Markdown
Owner

@Maples7 Maples7 commented May 16, 2026

Summary

  • clarify the vch land wrong-main-branch diagnostic to show the resolved target branch and current main-worktree branch
  • remove the confusing pass --into hint from that error path
  • add a regression test for the updated diagnostic

Tests

  • swift test --filter ErrorsTests/testLandMainNotOnIntoMessageNamesTargetAndCurrentBranch
  • swift test --parallel

Pre-commit review

  • /pre-commit-review: proceed, no findings

Closes #124

@Maples7 Maples7 merged commit 93bd6c8 into master May 16, 2026
1 check passed
@Maples7 Maples7 deleted the fix/124-land-into-diagnostic branch May 16, 2026 05:45
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.

vch land --into still requires main worktree to be on target branch

1 participant