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

Add changesets #126

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Add changesets #126

wants to merge 15 commits into from

Conversation

yamadapc
Copy link
Contributor

We want to publish non-canary/pre-release versions to NPM on a rolling basis.

Starting with a minor bump and adding the release script to the canary workflow. Not sure if there's more that needs to be done here.

@yamadapc yamadapc requested a review from a team October 15, 2024 00:13
release-command: yarn canary:release
release-command: |
yarn canary:release
yarn changeset publish
Copy link
Contributor

@MonicaOlejniczak MonicaOlejniczak Oct 15, 2024

Choose a reason for hiding this comment

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

Should we remove the lerna release then? Or are you moreso testing it out? Also, if we always publish on main then we should remove the tag-release workflow and associated commands. But, in saying that people might want a non-debug build too and having the generated pr for public releases is useful imo

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I will probably need to test this out a bit before being confident in it.

@yamadapc yamadapc marked this pull request as ready for review October 15, 2024 23:55
@yamadapc yamadapc force-pushed the issue/add-changesets branch from ae88d16 to 358e2e0 Compare October 16, 2024 04:52
@yamadapc yamadapc force-pushed the issue/add-changesets branch from 8c892b8 to fc170e3 Compare October 17, 2024 00:28
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