-
-
Notifications
You must be signed in to change notification settings - Fork 595
docs: remove alpha branch chapter from README #1430
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
Thanks for opening this pull request!
|
Thanks for pointing this out! I am not sure whether we want to keep that paragraph at all. We now have 2 branches (alpha, release) + in the future maybe beta + N LTS branches (release-N.x.x). To explain only the alpha branch seems a bit arbitrary. We actually have a better branch explanation in the changelog already, which is likely more prominent than that last paragraph in the README. To avoid keeping redundant info that has to maintained separately, I suggest we just remove the whole paragraph (and the TOC entry). |
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.
Looks good!
Codecov Report
@@ Coverage Diff @@
## alpha #1430 +/- ##
=======================================
Coverage 99.94% 99.94%
=======================================
Files 61 61
Lines 5948 5948
Branches 1354 1354
=======================================
Hits 5945 5945
Misses 3 3 Continue to review full report at Codecov.
|
🎉 This change has been released in version 3.4.0-alpha.3 |
🎉 This change has been released in version 3.4.1 |
New Pull Request Checklist
Issue Description
Update default branch name
Related issue: #1422
Approach
TODOs before merging