Skip to content

fix: auto-bump scanner releases after merge#38

Merged
kantorcodes merged 1 commit intomainfrom
feat/auto-release-versioning
Apr 3, 2026
Merged

fix: auto-bump scanner releases after merge#38
kantorcodes merged 1 commit intomainfrom
feat/auto-release-versioning

Conversation

@kantorcodes
Copy link
Copy Markdown
Member

Summary

  • derive the next patch version from the latest release tag on pushes to main
  • keep tagged releases publishing the explicit tag version unchanged
  • preserve the existing PyPI publish and GitHub release flow while removing the need for a manual source version bump before release

Verification

  • yq '.' .github/workflows/publish.yml >/dev/null
  • derived next version from latest tag: 1.4.1

Signed-off-by: Michael Kantor <6068672+kantorcodes@users.noreply.github.com>
@gemini-code-assist
Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@kilo-code-bot
Copy link
Copy Markdown

kilo-code-bot bot commented Apr 3, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 file)
  • .github/workflows/publish.yml - No issues

The implementation correctly:

  • Uses GITHUB_EVENT_NAME to differentiate between tag pushes and branch pushes
  • Parses the latest semantic version tag and increments only the PATCH component
  • Validates tag format before parsing to prevent malformed version strings
  • Falls back to BASE_VERSION (from pyproject.toml) when no tags exist, allowing manual override
  • Stamps the computed version into both pyproject.toml and version.py before building

The logic is sound and handles edge cases appropriately.


Reviewed by minimax-m2.5-20260211 · 191,062 tokens

@kantorcodes kantorcodes merged commit dce17b2 into main Apr 3, 2026
23 checks passed
@kantorcodes kantorcodes deleted the feat/auto-release-versioning branch April 3, 2026 18:56
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