-
Notifications
You must be signed in to change notification settings - Fork 3
[Feature] Changesets #3
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
Conversation
.github/workflows/dev-packages.yaml
Outdated
| @@ -0,0 +1,36 @@ | |||
| # Action to publish packages under the `next` tag for testing | |||
| # Packages are versioned as `0.0.0-dev.{short-sha}` | |||
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.
Is this comment still accurate? Changesets version pattern is slightly different
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.
Good catch! This has been updated to the new format
package.json
Outdated
| ] | ||
| }, | ||
| "dependencies": { | ||
| "@changesets/cli": "^2.26.2" |
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.
This can be moved to devDependencies seeing that it is only used on local dev and CI and not when the package is installed somewhere else
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.
Moved this now 🙏
Manrich121
left a 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.
Minor comments, but happy with the changes
This PR adds the Changesets tooling for this repo's package management.