-
Notifications
You must be signed in to change notification settings - Fork 22
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
[WIP] Add new branch checklist doc #188
base: master
Are you sure you want to change the base?
Conversation
6e0a8dd
to
32985ed
Compare
32985ed
to
7cbddb7
Compare
| `<NextBranch>` | Next branch name titleized | `Morphy` | | ||
| `<nextbranch#>` | Next branch number | `13` | | ||
| `<nextbranch>` | Next branch name | `morphy` | | ||
| `<n+2branch>` | Unnamed N+2 branch | `N-release` | |
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.
would it be more helpful if I have the n+2 branch name, so it is not "unnamed"?
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.
Yes
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.
The only place it's really used is here, which is stating when deprecations are expected to be removed.
Co-authored-by: Brandon Dunne <[email protected]>
Co-authored-by: Brandon Dunne <[email protected]>
Co-authored-by: Brandon Dunne <[email protected]>
ec2774a
to
1af4c5a
Compare
Co-authored-by: Brandon Dunne <[email protected]>
Once #215 is merged we can drop the entirety of section 4 |
@chessbyte Please review.
I keep this document locally, but it makes more sense to commit it here. WIP because I am mid-branching; some things are checked off and other parts are still incomplete. After branch is complete, I'll clean this up, clear all the checkboxes, and we can merge this.