Skip to content

Logic: Remove empty commits during rebase#309

Merged
OpsBotPrime merged 5 commits intomasterfrom
reinier/remove-empty-commit-during-rebase
Mar 27, 2026
Merged

Logic: Remove empty commits during rebase#309
OpsBotPrime merged 5 commits intomasterfrom
reinier/remove-empty-commit-during-rebase

Conversation

@ReinierMaas
Copy link
Copy Markdown

We are seeing empty commits from CoPilot being merged to master. We want to just drop these commits from the history when rebasing on top of master.

XRef: https://git-scm.com/docs/git-rebase#Documentation/git-rebase.txt---no-keep-empty

@ReinierMaas ReinierMaas self-assigned this Mar 27, 2026
@bertptrs bertptrs requested a review from Riscky March 27, 2026 13:41
Copy link
Copy Markdown

@Riscky Riscky left a comment

Choose a reason for hiding this comment

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

Thanks!

@ReinierMaas
Copy link
Copy Markdown
Author

CI disk is full. We have some magic incantations to fix this, which I will have to lookup.

@ReinierMaas
Copy link
Copy Markdown
Author

@OpsBotPrime merge!

ReinierMaas and others added 5 commits March 27, 2026 15:28
We are seeing empty commits from CoPilot being merged to master. We want to just drop these commits from the history when rebasing on top of master.

XRef: https://git-scm.com/docs/git-rebase#Documentation/git-rebase.txt---no-keep-empty
Approved-by: ReinierMaas
Priority: Normal
Auto-deploy: false
@OpsBotPrime
Copy link
Copy Markdown

Rebased as 797dd79, waiting for CI …

@OpsBotPrime
Copy link
Copy Markdown

CI job 🟡 started.

@OpsBotPrime OpsBotPrime force-pushed the reinier/remove-empty-commit-during-rebase branch from ce4a576 to 797dd79 Compare March 27, 2026 14:32
@OpsBotPrime OpsBotPrime merged commit 797dd79 into master Mar 27, 2026
1 check was pending
@OpsBotPrime OpsBotPrime deleted the reinier/remove-empty-commit-during-rebase branch March 27, 2026 14:32
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