Skip to content

Conversation

@DilumAluthge
Copy link
Member

@DilumAluthge DilumAluthge commented May 7, 2022

Instead of making the "no prereleases" check part of some other checks (like the "initial version" check and the "sequential version" check), this PR makes the "no prereleases" check its own top-level check.

@DilumAluthge DilumAluthge requested a review from giordano May 7, 2022 09:44
bors bot added a commit that referenced this pull request May 7, 2022
@DilumAluthge DilumAluthge marked this pull request as draft May 7, 2022 09:50
@JuliaRegistries JuliaRegistries deleted a comment from bors bot May 7, 2022
bors bot added a commit that referenced this pull request May 7, 2022
@DilumAluthge DilumAluthge marked this pull request as ready for review May 7, 2022 10:04
@JuliaRegistries JuliaRegistries deleted a comment from bors bot May 7, 2022
bors bot added a commit that referenced this pull request May 7, 2022
Copy link
Member

@giordano giordano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good as far as I can tell, but maybe adding a test would be useful

@DilumAluthge DilumAluthge force-pushed the dpa/version-numbers branch from 2ca68d5 to 6a64598 Compare May 8, 2022 06:24
@DilumAluthge
Copy link
Member Author

Added some tests

@bors bors bot merged commit 583e2bd into master May 8, 2022
@bors bors bot deleted the dpa/version-numbers branch May 8, 2022 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants