chore: drop gitmoji commit enforcement - #534
Merged
Nicola Franco (franconicola) merged 1 commit intoJul 26, 2026
Merged
Conversation
Remove the CI commit-check job and the local commitizen pre-commit hook, both of which required gitmoji-style commit messages via cz-conventional-gitmoji. Switch commitizen to plain conventional commits (still used for version bumping/changelog generation) and update CONTRIBUTING.md accordingly.
Copilot started reviewing on behalf of
Nicola Franco (franconicola)
July 26, 2026 20:23
View session
There was a problem hiding this comment.
Pull request overview
Removes repository-wide gitmoji-based commit message enforcement while keeping Commitizen for versioning/changelog workflows, aligning the project with plain Conventional Commits without CI/local hook gating.
Changes:
- Removed the CI “Commit Messages” job that ran
cz checkon PR commit ranges. - Removed the Commitizen
commit-msgpre-commit hook and droppedcz-conventional-gitmojifrom dependencies/lockfile. - Switched Commitizen configuration from
cz_gitmojitocz_conventional_commitsand updated contributing docs accordingly.
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
uv.lock |
Removes cz-conventional-gitmoji from the resolved dependency set (plus incidental lock marker updates from regeneration). |
pyproject.toml |
Drops the gitmoji adapter dependency and updates Commitizen to cz_conventional_commits. |
CONTRIBUTING.md |
Updates local setup instructions and removes references to commit-msg/CI enforcement for commit messages. |
.pre-commit-config.yaml |
Removes the Commitizen commit-msg hook configuration. |
.github/workflows/ci.yml |
Removes the commit-check job that enforced commit message format in CI. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Nicola Franco (franconicola)
enabled auto-merge
July 26, 2026 20:51
Nicola Franco (franconicola)
disabled auto-merge
July 26, 2026 20:52
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
commit-checkCI job (cz check) and the localcommitizenpre-commit hook, both of which required gitmoji-style commit messages viacz-conventional-gitmoji.[tool.commitizen]fromcz_gitmojitocz_conventional_commits(commitizen itself stays, for version bumping/changelog generation).CONTRIBUTING.mdto stop referencing the removed commit-msg hook and CI enforcement.Test plan
uv lockresolves cleanly after droppingcz-conventional-gitmoji.github/workflows/ci.ymland.pre-commit-config.yamlparse as valid YAML