-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add options for non-Python projects #65
Conversation
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #65 +/- ##
==========================================
- Coverage 95.53% 95.34% -0.19%
==========================================
Files 8 8
Lines 716 730 +14
==========================================
+ Hits 684 696 +12
- Misses 32 34 +2
☔ View full report in Codecov by Sentry. |
@@ -0,0 +1,12 @@ | |||
--- |
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.
I see this as two features. Why don't you split it in two changelog files? I would also do the same with the commits.
Adds the '--current-version' argument to 'semverup'. This uses the given version number instead of looking for it on '_version.py'. Signed-off-by: Eva Millán <[email protected]>
Adds the '--add-all' option to the 'publish' command to include all changed files in the release commit. Signed-off-by: Eva Millán <[email protected]>
66932e8
to
a5156f0
Compare
@sduenas I updated the PR with the changes you requested. |
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
This PR adds the
--current-version
arg to thesemverup
command and the--add-all
flag topublish
to avoid needing to use Python package specific files like_version.py
orpyproject.toml
.