Skip to content

ci(release): do not repeat a release that is already underway - #110

Merged
jdhughes-dev merged 1 commit into
MODFLOW-ORG:developfrom
jdhughes-dev:ci-release-guards
Sep 5, 2026
Merged

ci(release): do not repeat a release that is already underway#110
jdhughes-dev merged 1 commit into
MODFLOW-ORG:developfrom
jdhughes-dev:ci-release-guards

Conversation

@jdhughes-dev

Copy link
Copy Markdown
Contributor

Two defects in #108, both reachable while cutting 1.0.0.

  • Preparing a release triggered on any push to the release branch, so hand-editing the generated changelog there prepended a second section for the same version and failed trying to open a pull request that already existed. The release is now prepared only when the release branch is first pushed.
  • Skipping an already released version reported success, so a later push to main would attempt to upload the same version to PyPI again and open a second pull request resetting develop. The release job now reports whether it created a release, and the publish and reset jobs check it.

Preparing a release triggered on any push to the release branch, so
hand-editing the generated changelog on that branch prepended a second
section for the same version and failed trying to open a pull request that
already existed. The release is now prepared only when the release branch
is first pushed, leaving the branch open to corrections before it is merged.

Skipping an already released version reported success, so a later push to
main would attempt to upload the same version to PyPI again and open a
second pull request resetting develop. The release job now reports whether
it created a release, and the publish and reset jobs check it.
@jdhughes-dev
jdhughes-dev merged commit 6a31d10 into MODFLOW-ORG:develop Sep 5, 2026
34 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