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

chore: Fix beta release versioning #890

Merged
merged 1 commit into from
Jun 1, 2023
Merged

Conversation

emma-imber
Copy link
Contributor

What does this change?

Amends the grep command in the beta release script to return the latest release version tags.

Why?

The beta release versioning was using out of date versions because it was searching the git tag entries to find the latest versions. Now that bundle and core are merged, the release version tags are no longer prefixed with commerical-bundle or commercial-core in the tags, and are now just v8.0.0, for example. By removing the @guardian/commercial prefix from the grep command, we will find the latest tag version and assign the correct version number to the beta release.

Basically, tags used to look like this:
Screenshot 2023-06-01 at 14 32 18

And they now look like this:
Screenshot 2023-06-01 at 14 32 49

So we need to update the grep code accordingly.

@emma-imber emma-imber changed the title chore: Search for release tags without prefix chore: Fix beta release versioning Jun 1, 2023
@emma-imber emma-imber added the [beta] @guardian/commercial Add this label to publish an @guardian/commercial beta npm release from a PR label Jun 1, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jun 1, 2023

🚀 @guardian/commercial-v8.0.2-beta.0 published to npm as a beta release

@emma-imber emma-imber removed the [beta] @guardian/commercial Add this label to publish an @guardian/commercial beta npm release from a PR label Jun 1, 2023
Copy link
Contributor

@chrislomaxjones chrislomaxjones left a comment

Choose a reason for hiding this comment

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

Nice spot 👍

@emma-imber emma-imber marked this pull request as ready for review June 1, 2023 13:57
@emma-imber emma-imber requested a review from a team as a code owner June 1, 2023 13:57
Copy link
Member

@arelra arelra left a comment

Choose a reason for hiding this comment

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

🎉

@emma-imber emma-imber merged commit 5453e6f into main Jun 1, 2023
@emma-imber emma-imber deleted the ei/fix-beta-versioning branch June 1, 2023 14:06
@github-actions
Copy link
Contributor

github-actions bot commented Jun 8, 2023

🎉 This PR is included in version 8.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants