Skip to content

fix: prevent concurrent update-extension-list push failures#340

Merged
dotNomad merged 1 commit intomainfrom
dotnomad/isolate-update-extension-list
Apr 15, 2026
Merged

fix: prevent concurrent update-extension-list push failures#340
dotNomad merged 1 commit intomainfrom
dotnomad/isolate-update-extension-list

Conversation

@dotNomad
Copy link
Copy Markdown
Collaborator

@dotNomad dotNomad commented Apr 15, 2026

Summary

  • Adds a job-level concurrency group to the update-extension-list job so only one instance runs at a time (others queue instead of cancelling)
  • Adds git pull --rebase before git push as a safety net in case local main is behind remote

Fixes the race condition where merging multiple extension PRs in quick succession causes the second workflow run's push to be rejected (e.g. this run).

Add job-level concurrency group to serialize update-extension-list runs,
and add git pull --rebase before push as a safety net.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@dotNomad dotNomad requested review from lucasrod16 and tdstein April 15, 2026 17:08
Copy link
Copy Markdown
Collaborator

@jonkeane jonkeane left a comment

Choose a reason for hiding this comment

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

Thanks for this! I meant to come back to this and fix this, but got distracted elsewhere

@dotNomad dotNomad merged commit a927c33 into main Apr 15, 2026
12 checks passed
@dotNomad dotNomad deleted the dotnomad/isolate-update-extension-list branch April 15, 2026 18:42
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.

2 participants