Skip to content

[WIP] Improve bump pattern#1853

Open
namwoam wants to merge 5 commits intocommitizen-tools:masterfrom
namwoam:improve-bump-pattern
Open

[WIP] Improve bump pattern#1853
namwoam wants to merge 5 commits intocommitizen-tools:masterfrom
namwoam:improve-bump-pattern

Conversation

@namwoam
Copy link
Contributor

@namwoam namwoam commented Feb 7, 2026

Description

This PR will let user select commits that shouldn't be bump version. Solve #1772

Checklist

Was generative AI tooling used to co-author this PR?

  • Yes (please specify the tool below)

Code Changes

  • Add test cases to all the changes you introduce
  • Run uv run poe all locally to ensure this change passes linter check and tests
  • Manually test the changes:
    • Verify the feature/bug fix works as expected in real-world scenarios
    • Test edge cases and error conditions
    • Ensure backward compatibility is maintained
    • Document any manual testing steps performed
  • Update the documentation for the changes

Documentation Changes

  • Run uv run poe doc locally to ensure the documentation pages renders correctly
  • Check and fix any broken links (internal or external)

Expected Behavior

Steps to Test This Pull Request

Steps to reproduce the behavior:

  1. Add a test file test.txt
  2. commit "feat: this should not be bumped"
    suppose got the following git log:
commit e302cb4f2c626099b4a9c8cf881b0bb5906b7356
Author: andre.liang <namwoam@gmail.com>
Date:   Sat Feb 7 21:33:10 2026 +0800

    feat: add a new test file, this should not be bumped
  1. configure either ignore_bump_rev_list=["e302cb4f2c626099b4a9c8cf881b0bb5906b7356"] or ignore_bump_author_list=["andre.liang"] to ignore this commit from version bump calculation.
  2. run cz bump will produce:
[NO_COMMITS_TO_BUMP]
The commits found are not eligible to be bumped

Additional Context

@codecov
Copy link

codecov bot commented Feb 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.00%. Comparing base (573ffda) to head (486d9a0).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1853   +/-   ##
=======================================
  Coverage   97.99%   98.00%           
=======================================
  Files          60       60           
  Lines        2692     2702   +10     
=======================================
+ Hits         2638     2648   +10     
  Misses         54       54           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@namwoam namwoam marked this pull request as ready for review February 7, 2026 14:09
@namwoam
Copy link
Contributor Author

namwoam commented Feb 7, 2026

Hello @bearomorphism , this PR is ready. Please help me rename this PR to feat: implements ignore_bump_*, I don't have the permission to do that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant