-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
fix: Check all relevant pre-release versions, not just the latest. #346
Conversation
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.
Left a variety of comments. Some are things I think should be changed, others are suggestions I'm floating but am okay with having rejected.
4167f39
to
aeca427
Compare
BREAKING CHANGE: Handling of pre-release versions has been reworked to handle many more edge cases which results in more correct (but different) behavior. Closes #334
Co-authored-by: Michael Bryant <[email protected]>
Co-authored-by: Michael Bryant <[email protected]>
Co-authored-by: Michael Bryant <[email protected]>
BREAKING CHANGE: Handling of pre-release versions has been reworked to handle many more edge cases which results in more correct (but different) behavior. Closes #334
aeca427
to
e4ea303
Compare
@Shadow53 I think I fixed all the things, please take another look when you get a chance |
Merging this as-is since I think it fixes the bug. We can always improve the code more later! |
BREAKING CHANGE: Handling of pre-release versions has been reworked to handle many more edge cases which results in more correct (but different) behavior.
Closes #334