Skip to content

Coverage: publish onto the branch's current head - #15

Merged
JKRT merged 1 commit into
masterfrom
coverage-docs-rebase
Oct 29, 2026
Merged

JKRT merged 1 commit into
masterfrom
coverage-docs-rebase

Conversation

@JKRT

@JKRT JKRT commented Oct 28, 2026

Copy link
Copy Markdown
Member

A dispatched coverage run checks out the commit the branch had when it was queued. If another coverage run's docs commit lands while it waits (runs queue in the concurrency group), its docs push is rejected as a non-fast-forward and the site step is skipped. This is what would have happened tonight with 3.2.3 queued behind 4.1.0.

  • Publish: fetch the branch and take its current README.md before publish_coverage.jl writes this version's section.
  • Commit the documentation: git reset FETCH_HEAD, then commit only README.md and the version's coverage page on that head.

Tested by running both steps' run blocks from the workflow against a local origin with a depth-1 checkout. Master's workflow: push rejected. This branch: ok when the other run measured the other version (result identical to the real fed4dcd), the same version (this run's page and section win, nothing mixed), or when nothing landed in between.

🤖 Generated with Claude Code

A dispatched run checks out the commit the branch had when the run was
queued. When another coverage run's docs commit landed in the meantime
(runs wait in the concurrency group), the docs push was rejected as a
non-fast-forward and the site step skipped. Publish now writes the README
section into the branch's current README, and the docs commit goes on that
head with only README.md and the version's coverage page.

Assisted-by: Claude Opus 5.5
@JKRT
JKRT merged commit 11f98c6 into master Oct 29, 2026
2 checks passed
@JKRT
JKRT deleted the coverage-docs-rebase branch October 29, 2026 09:12
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