-
Notifications
You must be signed in to change notification settings - Fork 0
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
base: main
Are you sure you want to change the base?
Add changesets #126
Conversation
release-command: yarn canary:release | ||
release-command: | | ||
yarn canary:release | ||
yarn changeset publish |
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.
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
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.
I will probably need to test this out a bit before being confident in it.
ae88d16
to
358e2e0
Compare
8c892b8
to
fc170e3
Compare
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.