-
-
Notifications
You must be signed in to change notification settings - Fork 265
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: draft of the --empty parameter #723
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #723 +/- ##
==========================================
+ Coverage 97.33% 97.54% +0.21%
==========================================
Files 42 55 +13
Lines 2104 2489 +385
==========================================
+ Hits 2048 2428 +380
- Misses 56 61 +5
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
LGTM 💪🏻 , needs tests though |
🍾 Nice to see that you approve the idea.
absolutely, I'll work on them this week (I hope) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I recently happen to work on a project need such feature lol 🎉
@woile @noirbizarre, I picked this up and refactored it a bit. I changed the argument to |
thanks @Lee-W |
superceed #696
Description
First draft of a new feature. The idea is to grant the user the capability to run
cz bump
without prior commits. The reasons are described in the associated issue.here we add an
--empty
parameter to the bump methodwhen this parameter is set and if no increment is find, a automatic "PATCH" is released.
I'll wait for your feedback before working on tests and documentation.
Checklist
./scripts/format
and./scripts/test
locally to ensure this change passes linter check and testAdditional context
Fix #662