Skip to content

ci: enable auto-release with mcvs-general-action - #62

Open
sbp-bvanb wants to merge 1 commit into
mainfrom
ci-enable-auto-release
Open

sbp-bvanb wants to merge 1 commit into
mainfrom
ci-enable-auto-release

Conversation

@sbp-bvanb

@sbp-bvanb sbp-bvanb commented Oct 10, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Adds .github/workflows/release.yml following mcvs-general-action's docs/auto-release.md, pinned to v0.9.0.

Why: releases are tagged by hand today, so merged fixes and features wait until someone cuts a tag, and the version bump is a manual call. With this workflow every feat:/fix: merge is released automatically, with the version derived from the commit messages.

On every push to main it creates the next semver tag and a GitHub Release from the Conventional Commits since the latest tag (feat!:/BREAKING CHANGE: major, feat: minor, fix:/perf: patch, else nothing). The current latest tag is v0.3.0.

On merge: no release yet. main has no feat:/fix:/perf: commits since v0.3.0, and this PR's own ci: commit does not bump the version, so the next feat:/fix: merge cuts the first automatic release.

🤖 Generated with Claude Code

@sbp-bvanb sbp-bvanb left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

🤖 pr-reviewer report

Branch: ci-enable-auto-release → main | Head: de7895a | Commits: 1 | Files: 1
Verdict: ✅ Approved

Adds .github/workflows/release.yml. On every push to main, it runs mcvs-general-action v0.9.0 with auto-release: "true", which creates the next semver tag and a GitHub Release from the Conventional Commits.

Checks

  • Git: pass. Commit and PR title follow Conventional Commits. The description gives the reason and its claims hold: main is identical to v0.3.0, and the repo only allows squash merges, so this ci: title will not trigger a release.
  • Final newline: pass
  • Pinning: pass (4df2739… resolves to v0.9.0, where auto-release is a real input; actions/checkout uses the same SHA as the other workflows)
  • Correctness: pass. contents: write on the job only; concurrency serialises releases; the action does its own fetch-depth: 0 checkout. Matches the upstream snippet apart from the v0.9.0 pin and the # yamllint disable-line rule:line-length comment that general.yml and mcvs-pr-validation.yml also use.
  • Security: pass (persist-credentials: false, no secrets added)
  • Maintainability: pass
  • Over-engineering: pass
  • N/A: Terraform, Chef, Docker, Claude plugins

🤖 doc-reviewer report

Branch: ci-enable-auto-release → main | Head: de7895a | Docs: 0
Verdict: ❌ Changes requested

The PR adds only .github/workflows/release.yml. The root README.md it inherits from main has no ## Quickstart, and it carries the options reference and migration notes that belong in docs/, which does not exist.

Blocking

  • [quickstart] README.md:10: no ## Quickstart section; ## Usage already has the concrete steps (lines 12-30), so renaming its heading fixes this.

    ## Quickstart
    
  • [lean] README.md:32-70: the options table, usage notes and uv workspace example are reference material. Move them verbatim to a new docs/configuration.md under # Configuration.

  • [lean] README.md:72-87: "Migrating to v0.3.0" belongs in a new docs/migrating-to-v0.3.0.md. Replace it in the README with:

    ## Documentation
    
    - [Configuration](docs/configuration.md)
    - [Migrating to v0.3.0](docs/migrating-to-v0.3.0.md)
    

Checks

  • README: pass
  • Quickstart: fail (README.md:10)
  • Lean: fail (README.md:32-70, README.md:72-87)
  • docs/ linked: pass (0 pages)
  • Links: pass (2 checked)

🤖 dependency-reviewer report

Branch: ci-enable-auto-release → main | Head: de7895a | Dependencies: 3
Verdict: ✅ Approved

All pinned references in the new .github/workflows/release.yml are on their latest stable release, none are end-of-life, and Dependabot already covers the new workflow.

Checks

  • Up to date: 3 (actions/checkout@3d3c42e… # v7.0.1, schubergphilis/mcvs-general-action@4df2739… # v0.9.0, runs-on: ubuntu-24.04)
  • Outdated: 0
  • EOL: 0
  • Dependabot: the github-actions entry for directory: / has cooldown and groups

🤖 github-actions-reviewer report

Branch: ci-enable-auto-release → main | Head: de7895a | Actions: 2
Verdict: ✅ Approved

Checks

  • Pinning: pass (both uses: pinned to a full 40-character SHA; no local or docker:// actions)
  • Version comments: pass (v7.0.1 and v0.9.0, lightweight tags)
  • Version precision: pass

Generated by mcvs_review 3.5.0 (pr-reviewer, doc-reviewer, dependency-reviewer, github-actions-reviewer) at commit de7895a.

@sbp-bvanb

Copy link
Copy Markdown
Collaborator Author

Feedback from the mcvs_review summary:

  • README layout (doc-reviewer): not changed here. The README findings predate this PR, which only adds .github/workflows/release.yml; they belong in a separate docs PR.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ergousha
ergousha force-pushed the ci-enable-auto-release branch from de7895a to e96f7e5 Compare October 15, 2026 07:36
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.

2 participants