Skip to content

ci: migrate commit validation to prek+gitlint and standardize CI tool setup - #28

Merged
datenzar merged 5 commits into
mainfrom
ci/migrate-commit-checks-to-prek-gitlint
Mar 8, 2026
Merged

ci: migrate commit validation to prek+gitlint and standardize CI tool setup#28
datenzar merged 5 commits into
mainfrom
ci/migrate-commit-checks-to-prek-gitlint

Conversation

@datenzar

@datenzar datenzar commented Mar 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • replace Node/commitlint-based commit validation with prek + gitlint
  • standardize GitHub workflows around mise-managed Python/uv setup
  • update release automation to stage the generated uv.lock and publish artifacts through the existing build/test/release flow

What Changed

  • removed legacy commitlint/npm tooling in favor of repository-native commit validation
  • added CI support for commit-range validation through prek/gitlint
  • aligned workflows to install Python and uv via mise instead of mixed setup paths
  • updated .github/workflows/release.yml so release jobs:
    • reuse built distribution artifacts
    • download and stage uv.lock for the semantic-release version commit
    • keep build, e2e test, release, and PyPI deploy responsibilities clearly separated
  • refreshed contributor/agent guidance to match the enforced Conventional Commit and gitlint rules

Why

The repo was carrying overlapping tooling for commit validation and environment setup. This change reduces maintenance overhead, keeps local/CI behavior aligned, and makes the release pipeline more reproducible by ensuring the lockfile and build artifacts are handled consistently.

Testing

  • CI workflow updates only
  • release flow reviewed for build -> test-e2e -> release -> deploy handoff
  • commit validation path reviewed against CONTRIBUTING.md and enforced gitlint conventions

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3f5e002134

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread mise.toml
… setup

Replace Node/commitlint-based commit checks with prek+gitlint and add a mise task for CI commit-range validation.

Standardize GitHub workflows on mise-managed tool installation, add
PYTHON_VERSION-driven Python/uv setup, and remove obsolete Node
commitlint artifacts.

Update contributor/agent docs to reflect gitlint CT1 and commit body
line-length constraints.
@datenzar
datenzar force-pushed the ci/migrate-commit-checks-to-prek-gitlint branch from 3f5e002 to e3e7633 Compare March 8, 2026 22:05
datenzar added 4 commits March 8, 2026 23:09
Run the GitHub Actions commit-range checks through uvx so they do not depend on syncing the macOS-only project environment.

Keep local mise and prek commands on uv run so developer hooks still use the repository-managed dev environment.
Keep conventional title validation in CI while matching the repository's existing commit style, which often uses subject-only messages.
@datenzar
datenzar merged commit 35791d2 into main Mar 8, 2026
7 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