-
Notifications
You must be signed in to change notification settings - Fork 32
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
Automatically publish NPM package on every commit on main branch #758
Conversation
Affected libs:
|
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.
LGTM
I would test the artifact actions via a PR comment though, to be sure that it works as expected.
1406362
to
9b49677
Compare
9b49677
to
d433250
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.
Do you erase the commit i've pushed yesterday about the universal thing ?
oh yes sorry I forced push without pulling first, I'll readd it |
Readded here: #760 |
This should also publish the package on every release, as well as attempt a build of the npm package on every PR to make sure that it doesn't break.