Skip to content

docs(orphan): clarify help text mentions current-branch scope (#720) - #747

Merged
aviator-app[bot] merged 3 commits into
aviator-co:masterfrom
mvanhorn:docs/720-av-orphan-help-completeness
May 22, 2026
Merged

aviator-app[bot] merged 3 commits into
aviator-co:masterfrom
mvanhorn:docs/720-av-orphan-help-completeness

Conversation

@mvanhorn

Copy link
Copy Markdown
Contributor

Adds a Long description for av orphan covering current-branch scope, the trunk-orphans-everything behavior, and the av adopt recovery path.

Orphan branches that are managed by av

Usage:
  av orphan [flags]

Orphan the currently checked-out branch and any child branches that are managed by av.

When run from the trunk branch, this orphans every av-managed branch because all managed
branches descend from trunk. A separate warning for this behavior is tracked in #733.

To manage orphaned branches with av again, re-adopt them one at a time with "av adopt".

The branch-argument feature ask from #720 is intentionally out of scope here. Happy to file a follow-up issue or leave #720 open for tracking that.

Fixes #720

…r-co#720)

Adds Cobra Long description for av orphan that names current-branch scope,
cross-references the trunk-orphans-everything behavior tracked in aviator-co#733, and
points users at av adopt as the recovery path.
@mvanhorn
mvanhorn requested a review from a team as a code owner May 16, 2026 19:42
@aviator-app

aviator-app Bot commented May 16, 2026 •

Copy link
Copy Markdown
Contributor

Current Aviator status

Aviator will automatically update this comment as the status of the PR changes.
Comment /aviator refresh to force Aviator to re-examine your PR (or learn about other /aviator commands).

This PR was merged using Aviator (commit 2f295d6).


See the real-time status of this PR on the Aviator webapp.
Use the Aviator Chrome Extension to see the status of your PR within GitHub.

@aviator-app

aviator-app Bot commented May 16, 2026 •

Copy link
Copy Markdown
Contributor

✅ FlexReview Status

Common Owner: aviator-co/engineering (expert-load-balance assignment)
Owner and Assignment:

  • ✅ aviator-co/engineering (expert-load-balance assignment)
    Owned Files
    • 🔒 cmd/av/orphan.go

Review SLO: 7 business hours if PR size is <= 200 LOC for the first response.

@aviator-app
aviator-app Bot requested a review from jainankit May 16, 2026 19:42

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request adds a detailed long description to the orphan command in cmd/av/orphan.go, clarifying its impact on child branches and the trunk branch. The reviewer suggested removing an internal issue reference from the help text to keep documentation self-contained and correcting the description of the av adopt command to reflect its ability to handle multiple branches simultaneously.

Comment thread cmd/av/orphan.go Outdated
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@aviator-app
aviator-app Bot merged commit 2f295d6 into aviator-co:master May 22, 2026
5 checks passed
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.

av orphan help is incomplete.

2 participants