Skip to content

docs(advanced-git): merge-tip rebase collapse, no tail-piped gates, named staging, worktree .git file - #141

Merged
CybotTM merged 1 commit into
mainfrom
docs/b75-git-traps
Aug 5, 2026
Merged

docs(advanced-git): merge-tip rebase collapse, no tail-piped gates, named staging, worktree .git file#141
CybotTM merged 1 commit into
mainfrom
docs/b75-git-traps

Conversation

@CybotTM

@CybotTM CybotTM commented Aug 5, 2026

Copy link
Copy Markdown
Member

What

Four promoted traps (/retro promote, batch 7):

  • Plain rebase of a branch whose unique content sits in a merge commit silently collapses it (ahead = 0) and GitHub auto-closes the PR — verify ahead >= 1 after batch rebases, use --rebase-merges when merges carry content.
  • State-changing git/CLI commands piped through tail/head inside && chains report tail's exit code and hide the error — three tags on stale HEADs and one failed-but-"OK" verification build later, the rule is: gates run unpiped.
  • Never git add -A/. with legacy untracked files present — stage by name.
  • A worktree's .git is a FILE; --exclude-dir=.git doesn't protect it from bulk sed, which rewrites the gitdir pointer.

Also fixes this file's own cleanup example, which used the banned tail-pipe. The fifth note from the batch (worktree paths resolve from .bare) was already documented in detail and is tombstoned without changes.

Came from /retro: yes

…amed staging, worktree .git file

Four promoted traps; also removes a tail-pipe from this file's own
cleanup example (the pattern the new section bans).

Signed-off-by: Sebastian Mendel <info@sebastianmendel.de>
Copilot AI lite review requested due to automatic review settings August 5, 2026 20:09
@sonarqubecloud

sonarqubecloud Bot commented Aug 5, 2026

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@github-actions github-actions Bot added documentation Improvements or additions to documentation skill labels Aug 5, 2026

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 reached their quota limit.

@CybotTM
CybotTM merged commit 987d4e5 into main Aug 5, 2026
21 of 22 checks passed
@CybotTM
CybotTM deleted the docs/b75-git-traps branch August 5, 2026 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation skill

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants