Skip to content

Mark branches that need a restack in log short - #63

Merged
DomBlack merged 2 commits into
DomBlack:mainfrom
mvalipour:log-short-restack-marker
Oct 29, 2026
Merged

DomBlack merged 2 commits into
DomBlack:mainfrom
mvalipour:log-short-restack-marker

Conversation

@mvalipour

Copy link
Copy Markdown
Contributor

The names-only tree hid the one thing a stack view is most useful for: whether a branch has fallen behind its parent. Short now appends "(needs restack)" to those branches. The check only reads local git, so short still never touches the network and stays quick.

The restack check was skipped for short before. It is now run for it as well, and the README, help text and rendering tests are updated to match.

The names-only tree hid the one thing a stack view is most useful for:
whether a branch has fallen behind its parent. Short now appends
"(needs restack)" to those branches. The check only reads local git, so
short still never touches the network and stays quick.

The restack check was skipped for short before. It is now run for it as
well, and the README, help text and rendering tests are updated to match.
Copilot AI balanced review requested due to automatic review settings October 28, 2026 18:21

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@DomBlack
DomBlack self-requested a review October 28, 2026 18:28
…hole

The renderer only redraws what changed, so after typing "hot" the last
frame can be a cursor move and a lone "t"; "> hot" never appears in the
output and the wait times out (seen on CI under -race). Messages are
handled in order, so enter already sees the whole filter without waiting.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 28, 2026 18:34

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@DomBlack
DomBlack merged commit dcc7bd5 into DomBlack:main Oct 29, 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.

3 participants