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

[STA-3429] pre-release: do not apply pre-release tag to 0.x.x versions #59

Merged
merged 2 commits into from
Jan 29, 2024

Conversation

meorphis
Copy link

Summary

Makes it so that we do not apply the prerelease GH tag to 0.x.x versions

Test Plan

Update existing test for this case

@meorphis meorphis requested a review from dgellow January 25, 2024 18:06
@meorphis meorphis changed the title prerelease: support format that includes a dot pre-release: do not apply tag to 0.x.x versions Jan 25, 2024
@meorphis meorphis changed the title pre-release: do not apply tag to 0.x.x versions pre-release: do not apply pre-release tag to 0.x.x versions Jan 25, 2024
@meorphis meorphis changed the title pre-release: do not apply pre-release tag to 0.x.x versions [STA-3429] pre-release: do not apply pre-release tag to 0.x.x versions Jan 25, 2024
@@ -6908,7 +6908,7 @@ version = "3.0.0"
);
});

it('should build prerelease releases from pre-major', async () => {
it('should not build prerelease releases from pre-major', async () => {
Copy link
Member

Choose a reason for hiding this comment

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

Do we have a test case that covers 0.x.y AND the label autorelease: pre-release? To be sure users have a way to overwrite that behavior if they want a specific release to be tagged as pre-release.

Copy link
Author

Choose a reason for hiding this comment

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

added a test

@meorphis meorphis merged commit 6c9ab11 into main Jan 29, 2024
4 checks passed
@meorphis meorphis deleted the emorphis-0-versions-not-prerelease branch January 29, 2024 18:37
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.

2 participants