-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Remove auto-publishing on develop; add canary tool #954
Conversation
|
|
||
on: | ||
# enable users to manually trigger with workflow_dispatch | ||
workflow_dispatch: {} |
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.
yarn changeset version --snapshot | ||
yarn changeset publish --tag canary |
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.
@@ -3,7 +3,6 @@ name: Release | |||
on: | |||
push: | |||
branches: | |||
- develop |
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.
After this, we will only publish real versions once we merge to master
like this
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.
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
This pull request does two things:
develop
npm
versions and Docker images at a snapshot given a branch. For example:0.0.0-canary-127312312
may be the tag name for every package.(1) solves the issue of us needing to make changes to our code before we merge to
master
, which is true "stable" branch. (2) gives us the tool we need once (1) is done to still be able to test onk8s
and other production-like environments.