feat: add --allow-no-commit option

[12rambau]

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 method
when 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

• 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

Additional context

Fix#662

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.59%. Comparing base (120d514) to head (10740c9).
Report is 551 commits behind head on master.

Additional details and impacted files

@@ Coverage Diff @@## master #723 +/- ##
==========================================
+ Coverage 97.33% 97.59% +0.25% 
==========================================
Files 42 55 +13 Lines 2104 2617 +513 ==========================================
+ Hits 2048 2554 +506 - 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.

[woile]

LGTM 💪🏻 , needs tests though

[12rambau]

🍾 Nice to see that you approve the idea.

needs tests though

absolutely, I'll work on them this week (I hope)

[Lee-W]

LGTM. I recently happen to work on a project need such feature lol 🎉

[Lee-W]

@woile@noirbizarre, I picked this up and refactored it a bit. I changed the argument to --allow-no-commit. One weird thing that might happen is that one commit might have multiple versions. I'm not sure whether it would cause any issue

[12rambau]

thanks @Lee-W

[Lee-W]

Hey @woile@noirbizarre , I just rebased it from the latest master branch. Would be nice if you could take a look. Thanks!

[woile]

Should this be exposed as a config parameter as well? besides that it looks good

[Lee-W]

Should this be exposed as a config parameter as well? besides that it looks good

Yep, could be. let me update it as well. Just notice the CI also failed 🤦‍♂️

[Lee-W]

Should this be exposed as a config parameter as well? besides that it looks good

@woile After a second thought, I think we probably shouldn't add it. I feel this is somewhat like an exception, and if we're to do it, we probably need to introduce a better config/arugment handling to avoid #996

[woile]

And there's still an issue with vsc with no commits

[Lee-W]

And there's still an issue with vsc with no commits

I'm afraid I don't quite get this one 🤔

[noirbizarre]

I'll take a look tomorrow 👍🏽

[noirbizarre]

LGTM ! (before it reaches 2 years old, sorry for that)
This feature is needed. I think we can iterate on smaller pull request to add later settings or any other adjustments