-
Notifications
You must be signed in to change notification settings - Fork 332
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
fix: update publish command and add notice to raise PR against next branch #3013
Conversation
01b45c3
to
832f7cc
Compare
832f7cc
to
d061af0
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.
Just a comment regarding the text
d061af0
to
8d99a98
Compare
d590ba2
to
6f31116
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.
In order for this to publish within a workspace environment, the command on each package must used yarn
:
yarn npm publish
6f31116
to
953df78
Compare
Done |
COMPLETES #<SPARK-449056>
This pull request addresses
package.json
file under packagesmaster
branch will be frozen and to raise all the PRs againstnext
branchby making the following changes
Change Type
The following scenarios where tested
< ENUMERATE TESTS PERFORMED, WHETHER MANUAL OR AUTOMATED >
I certified that
I have read and followed contributing guidelines
I discussed changes with code owners prior to submitting this pull request
I have not skipped any automated checks
All existing and new tests passed
I have updated the documentation accordingly
Make sure to have followed the contributing guidelines before submitting.