-
-
Notifications
You must be signed in to change notification settings - Fork 62
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
cleanup(gittag, githubrelease) remove deprecated code for field version
#590
Conversation
version
6ee7d03
to
ec8a862
Compare
- Deprecated field "version" - updatecli/updatecli#590 - Deprecated fields "postfix/prefix" - updatecli/updatecli#590 - Deprecated fields "replacers" - updatecli/updatecli#592 - SCM setup moved to the top level directive scms - updatecli/updatecli#388 - Deprecated top-level field source (singular source) - updatecli/updatecli#589 Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
ec8a862
to
f2c4ef5
Compare
…testing Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
…rsion.Filter is used instead) Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
f2c4ef5
to
1deac50
Compare
|
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.
Thanks for this huge cleanup, especially considering that I failed to correctly document the code which evolved many times
Thanks, but don't be too harsh with yourself :) It's a looot of work, so let's improve this tool and its doc one step after the other |
- Deprecated field "version" - updatecli/updatecli#590 - Deprecated fields "postfix/prefix" - updatecli/updatecli#590 - Deprecated fields "replacers" - updatecli/updatecli#592 - SCM setup moved to the top level directive scms - updatecli/updatecli#388 - Deprecated top-level field source (singular source) - updatecli/updatecli#589 Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
This PR removes the deprecated code of the field
version
.It also introduces the following changes:
Makefile
to allow executing the end to end test with a custom binary directory (e.g. allowmake test-e2e VENOM_VAR_binpath=./custom-dist/
to use the binaryupdatecli
located in./custom-dist/updatecli
)Init()
(interface ScmHandler) as it was only used to define versionfilters.Associated documentation PR: updatecli/website#264
Test
To test this pull request, you can run the following commands:
make test
Additional Information
Tradeoff
Potential improvement