Skip to content
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

Merged
merged 8 commits into from
Jan 16, 2023

Conversation

dbanty
Copy link
Member

@dbanty dbanty commented Dec 19, 2022

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

@dbanty
Copy link
Member Author

dbanty commented Dec 19, 2022

This replaces #335@Shadow53 will you review this please?

@dbanty dbanty requested a review from Shadow53 December 19, 2022 22:57
@dbanty dbanty added 🐛 bug Something isn't working 🥚 breaking change This change will require existing users to change something about their config labels Dec 19, 2022
Copy link
Collaborator

@Shadow53 Shadow53 left a 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.

@dbanty dbanty force-pushed the 334-handle-pre-too-new branch from 4167f39 to aeca427 Compare January 6, 2023 21:31
dbanty and others added 6 commits January 6, 2023 14:31
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]>
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
@dbanty dbanty force-pushed the 334-handle-pre-too-new branch from aeca427 to e4ea303 Compare January 6, 2023 21:32
@dbanty
Copy link
Member Author

dbanty commented Jan 6, 2023

@Shadow53 I think I fixed all the things, please take another look when you get a chance

@dbanty
Copy link
Member Author

dbanty commented Jan 16, 2023

Merging this as-is since I think it fixes the bug. We can always improve the code more later!

@dbanty dbanty merged commit 53a552c into main Jan 16, 2023
@dbanty dbanty deleted the 334-handle-pre-too-new branch January 16, 2023 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working 🥚 breaking change This change will require existing users to change something about their config
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider "too new" prerelease tags when bumping.
2 participants