-
Notifications
You must be signed in to change notification settings - Fork 0
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
Implement support for key revision.tag=last #25
Conversation
…est and let the tool attempt to parse out all the tags from the git repo and determine what tag should be used and if we need to update. There are additional options to set to this revision.tag=latest feature, see docs for details about each individual option.
Co-authored-by: Henrik Holmboe <henrik@dynamist.se>
…e. Add __all__ to exceptions.py and constants.py to make them properly importable with *
…sing, filter, order & select. - Replace package semver with packaging for better PEP440 support - Separate out filter, order & select to new methods to declutter update() method - Impelment unittests for new logics to validate each step and the entire chain works together as expected - New enums.py file that will be used to define all supported algorithms we have
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.
Minor comment. LGTM.
|
||
class SelectionMethods(Enum): | ||
SEMVER = 10 | ||
EXACT = 20 |
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.
Maybe use LITERAL
instead of EXACT
? You decide.
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 might, but let me think about it and i will push it in a future commit if i change it. I find both options a little bit wonky tbh
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 added in this as a idea card in the 1.0.0 project for keeping track of this suggestion
Closes #23
Implement support for specifying "revision.tag=latest" key and support for a few helper options that can be used to filter and clean tags to allow for semver to work it's magic. Please see docs for additional examples and usage of all options.