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(release): [1.2] verifying plugins have been released to npmjs.com and bumping versions #2350

Closed

Conversation

Fortune-Ndlovu
Copy link
Member

@Fortune-Ndlovu Fortune-Ndlovu commented Oct 11, 2024

Issue: https://issues.redhat.com/browse/RHIDP-4217

Bumping plugin versions based on the npm registry and previous release.

@Fortune-Ndlovu Fortune-Ndlovu requested a review from a team as a code owner October 11, 2024 16:05
@Fortune-Ndlovu Fortune-Ndlovu changed the title chore(deps) verifying plugins have been released to npmjs.com and bumping versions chore(release): [1.3] verifying plugins have been released to npmjs.com and bumping versions Oct 11, 2024
@Fortune-Ndlovu Fortune-Ndlovu changed the title chore(release): [1.3] verifying plugins have been released to npmjs.com and bumping versions chore(release): [1.2] verifying plugins have been released to npmjs.com and bumping versions Oct 11, 2024
@@ -1,6 +1,6 @@
{
"name": "@janus-idp/backstage-plugin-acr",
"version": "1.4.11",
"version": "1.7.11",
Copy link
Member

Choose a reason for hiding this comment

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

the version in the 1.3 branch is ALSO at 1.7.11 so this will collide. https://github.com/janus-idp/backstage-plugins/blob/release-1.3/plugins/acr/package.json#L3

however 1.7.z is the correct version for the 1.3 branch, as main has moved ACR to https://github.com/backstage/community-plugins/commits/main/workspaces/acr/plugins/acr/package.json where it's at version 1.8.z

Should this be at 1.6.z in 1.2?

Suggested change
"version": "1.7.11",
"version": "1.6.4",

Copy link
Member Author

Choose a reason for hiding this comment

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

I missed the version in other branches, your right that makes sense based on the current versioning across the branches. That is:

  • main branch 1.8.z (already moved to community-plugins repo)
  • release-1.3 branch 1.7.z (currently at 1.7.11, which is correct for this branch)
  • 1.2.x branch Since 1.7.11 is used in 1.3, and 1.8.z is in the main branch, the 1.2.z branch should have a version that's lower than 1.7.z to avoid conflicts (Logically speaking, your right). Hence 1.6.4 in 1.2.z

@Fortune-Ndlovu
Copy link
Member Author

cc: @nickboldt / @kim-tsao for review

Copy link

changeset-bot bot commented Oct 17, 2024

⚠️ No Changeset found

Latest commit: 4dd2eac

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@Fortune-Ndlovu
Copy link
Member Author

closed because i have a pr targetting all

Copy link

sonarcloud bot commented Nov 5, 2024

@Fortune-Ndlovu Fortune-Ndlovu closed this by deleting the head repository Nov 8, 2024
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