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: Implement maintenance release config. #373

Merged
merged 3 commits into from
Feb 8, 2024

Conversation

kinyoklion
Copy link
Member

@kinyoklion kinyoklion commented Feb 8, 2024

This implements support for a maintenance branch for creating node-server-sdk releases.

It removes all builds related to other packages and can only build/release node-server-sdk.

yarn workspace $WORKSPACE npm publish || { echo "npm publish failed" >&2; exit 1; }
# Using a maintenance tag prevents this release from getting tagged with "latest".
echo "Publishing with maintenance tag."
yarn workspace $WORKSPACE npm publish --tag maintenance || { echo "npm publish failed" >&2; exit 1; }
Copy link
Member Author

Choose a reason for hiding this comment

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

If you do not include a tag, then it defaults to latest. This results in npm i @launcharkly/node-server-sdk installing the maintenance release. (This was a problem with releaser).

@kinyoklion kinyoklion marked this pull request as ready for review February 8, 2024 22:01
Copy link
Contributor

@yusinto yusinto left a comment

Choose a reason for hiding this comment

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

I'm approving, but a little disappointed RP does not support this in a more elegant fashion.

@kinyoklion
Copy link
Member Author

I'm approving, but a little disappointed RP does not support this in a more elegant fashion.

Well, it may, but it depends what is meant by elegant. This version is careful. I could just branch and change the branch name.

@kinyoklion kinyoklion merged commit 1b7fe3c into node-server-sdk-8.x Feb 8, 2024
4 checks passed
@kinyoklion kinyoklion deleted the rlamb/node-server-8.x-maintenance branch February 8, 2024 22:25
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