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: Publish new version reverting 59.5.3 #2484

Merged
merged 1 commit into from
Oct 21, 2024
Merged

Conversation

AshCorr
Copy link
Member

@AshCorr AshCorr commented Oct 18, 2024

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.

Copy link

changeset-bot bot commented Oct 18, 2024

🦋 Changeset detected

Latest commit: 08e6833

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@guardian/cdk Patch

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",
Copy link
Member Author

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.

Comment on lines +3 to +6
## 59.5.3

**DO NOT USE**: Published by mistake.

Copy link
Member Author

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.

Comment on lines 7 to 10
- 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
Copy link
Member

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.

Copy link
Member Author

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!

@AshCorr AshCorr merged commit 9a2db96 into main Oct 21, 2024
4 checks passed
@AshCorr AshCorr deleted the ash/unpublish-59.5.3 branch October 21, 2024 12:40
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