Skip to content

fix: give the pipeline its own concurrency group to avoid a publish deadlock - #32

Merged
GoogilyBoogily merged 1 commit into
mainfrom
DTBTWEB-1765-fix-concurrency-deadlock
Aug 19, 2026
Merged

fix: give the pipeline its own concurrency group to avoid a publish deadlock#32
GoogilyBoogily merged 1 commit into
mainfrom
DTBTWEB-1765-fix-concurrency-deadlock

Conversation

@GoogilyBoogily

Copy link
Copy Markdown
Contributor

release-pipeline.yml and the publish.yml it calls both used the npm-publish concurrency group. A nested workflow sharing its caller's group deadlocks, so every publish run got canceled.

Renames the pipeline's group to npm-release. publish.yml keeps npm-publish, so the actual npm-publish step still serializes across both the pipeline and standalone runs, with no parent/child clash.

@GoogilyBoogily
GoogilyBoogily requested review from a team and a lite review from Copilot August 19, 2026 19:19

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.

Pull request overview

Updates the GitHub Actions release pipeline workflow to avoid a concurrency deadlock when it calls the nested publish.yml reusable workflow by giving the pipeline its own distinct concurrency group.

Changes:

  • Change .github/workflows/release-pipeline.yml concurrency group from npm-publish to npm-release to prevent parent/child workflow concurrency group collisions.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@RickeySerna RickeySerna 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.

🛺

@adrmachado-public adrmachado-public left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lol

@GoogilyBoogily
GoogilyBoogily merged commit fdce883 into main Aug 19, 2026
2 checks passed
@GoogilyBoogily
GoogilyBoogily deleted the DTBTWEB-1765-fix-concurrency-deadlock branch August 19, 2026 19:34
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.

4 participants