-
Notifications
You must be signed in to change notification settings - Fork 6
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: Publish new version reverting 59.5.3 #2484
Conversation
🦋 Changeset detectedLatest commit: 08e6833 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
@@ -1,7 +1,7 @@ | |||
{ | |||
"name": "@guardian/cdk", | |||
"description": "Generic Guardian flavoured AWS CDK components", | |||
"version": "59.5.2", | |||
"version": "59.5.3", |
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.
Don't let it try to publish to 59.5.3 when it generates a release.
## 59.5.3 | ||
|
||
**DO NOT USE**: Published by mistake. | ||
|
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.
Add a changelog entry for 59.5.3 to let people know why it exists and to not use it.
.changeset/tall-eagles-heal.md
Outdated
- a334ea9: bump @types/node from 22.7.4 to 22.7.5 | ||
- 504fe94: bump @changesets/cli from 2.27.8 to 2.27.9 | ||
- 5a5498d: bump @types/node from 22.7.0 to 22.7.4 | ||
- 7dbd8ac: bump @types/node from 22.5.5 to 22.7.0 |
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.
These are all dev dependencies; I'm surprised to see them causing a version change.
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.
Ah yea, good point! Removed!
2da8d36
to
08e6833
Compare
What does this change?
v59.5.3 was published by mistake whilst trying to publish a beta build from the
beta
branch.Instead of unpublishing 59.5.3 which would leave a hole in our NPM versions as per https://docs.npmjs.com/policies/unpublish instead publish a new 59.5.4 version which reverts the changes that were made in 59.5.3.
Also include the latest version bumps.