-
Notifications
You must be signed in to change notification settings - Fork 22
add @changesets/changelog-github for better changelogs #589
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
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ |
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 16fbce8:
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
7b9808b
to
593cda1
Compare
.github/workflows/release.yml
Outdated
uses: changesets/action@06245a4e0a36c064a573d4150030f5ec548e4fcc # v1.4.10 | ||
with: | ||
version: pnpm run changeset version # Updates versions and changelog | ||
# publish: pnpm run changeset publish # Publishes to npm |
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.
Do we want to do anything with this comment?
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.
updated this to be used in the publish flow
.github/workflows/release.yml
Outdated
# publish: pnpm run changeset publish # Publishes to npm | ||
env: | ||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # For changelog links | ||
# NPM_TOKEN: ${{ secrets.NPM_TOKEN }} # For npm 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.
And this one?
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.
updated this to be used in the publish flow
5e33866
to
0321b4a
Compare
8fc32db
to
833f930
Compare
Summary & Motivation
Created a release workflow via GitHub Action which will
@turnkey/sdk-types
for the time being as we test the workflowenvironment: production
flag will pause the workflow at the publish step and will require a manual approval before performing the actual publish stepWith @changesets/changelog-github, it includes GitHub PR and author links if run in CI with a GITHUB_TOKEN.
The new changelog format will look like this:
How I Tested These Changes
Did you add a changeset?
If updating one of our packages, you'll likely need to add a changeset to your PR. To do so, run
pnpm changeset
.pnpm changeset
will generate a file where you should write a human friendly message about the changes. Note how this (example) includes the package name (should be auto added by the command) along with the type of semver change (major.minor.patch) (which you should set).These changes will be used at release time to determine what packages to publish and how to bump their version. For more context see this comment.