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

ci: update release config to publish to npm #41

Merged
merged 3 commits into from
Sep 2, 2022

Conversation

devisscher
Copy link
Contributor

This PR is updating the release configuration to publish the package directly to NPM when a release is created using changesets.

  1. Changesets will be generated on feature branches
  2. A release PR is generated and merged when a release is to be cut
  3. The release PR triggers the release action which runs the changeset publish.

@devisscher devisscher requested a review from a team September 2, 2022 19:59
@@ -5,3 +5,37 @@
### Patch Changes
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This file was CHANGELOG.MD - but git doesn't play well with case sensitive filenames. So I'm deleting the the capital cased file extension.

Copy link
Contributor

@knames knames left a comment

Choose a reason for hiding this comment

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

I don't have a whole lot of experience with npm releases, but at the surface level this seems good to me!

@devisscher devisscher merged commit 31abc78 into main Sep 2, 2022
@devisscher devisscher deleted the update-release-config branch September 2, 2022 20:10
@github-actions github-actions bot mentioned this pull request Sep 2, 2022
@devisscher
Copy link
Contributor Author

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.

None yet

2 participants