Skip to content

Prevent merged PR cleanup from cancelling docs deployment - #284

Merged
uriahf merged 1 commit into
mainfrom
agent/serialize-docs-deployments
Aug 11, 2026
Merged

Prevent merged PR cleanup from cancelling docs deployment#284
uriahf merged 1 commit into
mainfrom
agent/serialize-docs-deployments

Conversation

@uriahf

@uriahf uriahf commented Aug 11, 2026

Copy link
Copy Markdown
Owner

What changed

  • use one shared concurrency group for documentation writes
  • queue documentation deployments instead of cancelling an in-progress deployment

Why

When a PR is merged, GitHub emits both:

  1. a push to main, which builds and publishes production docs
  2. a closed pull-request event, which removes the PR preview

For a merged PR, both runs resolve to the same previous concurrency key. Because cancel-in-progress was enabled, the preview-cleanup run cancelled the production deployment. This happened after both PR #282 and PR #283.

Serializing the runs prevents concurrent writes to gh-pages and ensures neither production publishing nor preview cleanup is dropped.

Validation

  • package CI after PR Separate PyPI publishing from package CI #283 passed
  • the cancelled production run and successful closed-event cleanup started at the same timestamp
  • this change is limited to the documentation workflow concurrency policy

@github-actions

github-actions Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-08-11 03:43 UTC

@uriahf
uriahf merged commit 3fa182b into main Aug 11, 2026
3 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.

1 participant