Skip to content

Conversation

@carlossg
Copy link
Contributor

@carlossg carlossg commented Feb 7, 2025

command

To help understand why increments happen

Description

Checklist

  • Add test cases to all the changes you introduce
  • Run ./scripts/format and ./scripts/test locally to ensure this change passes linter check and test
  • Test the changes on the local machine manually
  • Update the documentation for the changes

Expected behavior

Steps to Test This Pull Request

Additional context

@carlossg carlossg marked this pull request as ready for review February 7, 2025 13:34
@codecov
Copy link

codecov bot commented Feb 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.59%. Comparing base (120d514) to head (80bbdb2).
Report is 558 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1353      +/-   ##
==========================================
+ Coverage   97.33%   97.59%   +0.25%     
==========================================
  Files          42       55      +13     
  Lines        2104     2622     +518     
==========================================
+ Hits         2048     2559     +511     
- Misses         56       63       +7     
Flag Coverage Δ
unittests 97.59% <100.00%> (+0.25%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Copy link
Member

@Lee-W Lee-W left a comment

Choose a reason for hiding this comment

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

LGTM. do we want to make it release a new version or just for easy developing

@Lee-W Lee-W added pr-status: ready-to-merge almost ready to merge. just keep it for a few days for others to check and removed pr-status: wait-for-review labels Feb 8, 2025
@carlossg
Copy link
Contributor Author

carlossg commented Feb 8, 2025

I personally don't need a new release, thanks!

@Lee-W Lee-W merged commit fe3b726 into commitizen-tools:master Feb 8, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-status: ready-to-merge almost ready to merge. just keep it for a few days for others to check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants