-
Notifications
You must be signed in to change notification settings - Fork 1
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
Introduce Changesets 🎉 #338
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
85cec76
to
109a36b
Compare
109a36b
to
f919311
Compare
f919311
to
dea1922
Compare
- name: Grab the released version and store it in an env variable | ||
run: | | ||
VERSION=$(jq -r '.version' ./packages/mantle/package.json) | ||
echo "MANTLE_VERSION=$VERSION" >> $GITHUB_ENV |
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.
We get outputs from the changesets action, so we can remove this step 🙂 https://github.com/changesets/action?tab=readme-ov-file#outputs
|
||
jobs: | ||
publish: | ||
runs-on: ubuntu-latest | ||
if: github.repository == 'ngrok-oss/mantle' |
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 think technically we should have this guard, since we're a public repo that could get forked.
dea1922
to
d7e093f
Compare
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.
let's give it a whirl and fix anything (including gameplan/recipes/documentation) in a follow up
Introduce Changesets to help us manage versioning/publishing/changelogs.