Skip to content

ci: fail loudly on malformed landed release subjects - #37

Merged
rianjs merged 1 commit into
mainfrom
ci/36-fail-loud-release-gate
Aug 3, 2026
Merged

ci: fail loudly on malformed landed release subjects#37
rianjs merged 1 commit into
mainfrom
ci/36-fail-loud-release-gate

Conversation

@rianjs

@rianjs rianjs commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • distinguish valid non-release conventional subjects from malformed landed subjects in the shared release-gate checker
  • preserve feat/fix release eligibility and docs/refactor/ci skip behavior
  • add regression coverage for release, skip, and fail-loud cases

Root cause

The auto-release action treated every release-gate exit 1 as an intentional skip. A non-conventional landed subject could therefore green-skip auto-release even when the PR title was conventional. Malformed subjects now return an error status with a targeted message, which the existing action wrapper fails loudly.

Checks

  • bash actions/auto-release/test_gate.sh
  • bash -n actions/conventional-commit/check.sh
  • bash -n actions/auto-release/test_gate.sh
  • actionlint
  • git diff --check

The repository Python test command could not run locally because the environment has no installed pytest module; the workflow installs its pinned test dependencies.

Closes #36

@rianjs
rianjs merged commit 7653eae into main Aug 3, 2026
20 checks passed
@rianjs
rianjs deleted the ci/36-fail-loud-release-gate branch August 3, 2026 11: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.

Auto-release must fail loudly on malformed landed commit subjects

1 participant