-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Update release process and master
becomes the stable branch
#4719
Conversation
Let's also update https://github.com/storybooks/storybook/blob/master/.github/PULL_REQUEST_TEMPLATE.md with a note that PR base shouldn't be changed from |
Codecov Report
@@ Coverage Diff @@
## master #4719 +/- ##
=======================================
Coverage 35.55% 35.55%
=======================================
Files 557 557
Lines 6738 6738
Branches 885 885
=======================================
Hits 2396 2396
Misses 3881 3881
Partials 461 461 Continue to review full report at Codecov.
|
@Hypnosphi updated |
.github/PULL_REQUEST_TEMPLATE.md
Outdated
|
||
If your answer is yes to any of these, please make sure to include it in your PR. | ||
|
||
For maintainers only: Please tag your pull request with at least one of the following: | ||
**For everybody:** Please submit all PRs to the `next` branch unless they are specific to current release. Storybook maintainers cherry-pick bug and documentation fixes into the `master` branch as part of the release process, so you shouldn't need to worry about 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.
let's surround both notes with <!--
/-->
to hide them from actual PRs
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.
Issue: N/A
What I did
Update
master
branch per the release process v4 proposalHow to test
Is this testable with Jest or Chromatic screenshots?
Does this need a new example in the kitchen sink apps?
Does this need an update to the documentation?
If your answer is yes to any of these, please make sure to include it in your PR.
For maintainers only: Please tag your pull request with at least one of the following:
["cleanup", "BREAKING CHANGE", "feature request", "bug", "documentation", "maintenance", "dependencies", "other"]