-
Notifications
You must be signed in to change notification settings - Fork 266
Document process for bumping dev-master branch-alias and bump to 1.10-dev #852
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
CONTRIBUTING.md
Outdated
@@ -124,9 +124,38 @@ The maintenance branch's HEAD will be the target for our release tag: | |||
$ git tag -a -m "Release X.Y.Z" X.Y.Z | |||
``` | |||
|
|||
### Push tags | |||
### Update branch-alias for dev-master (major and minor releases only) |
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.
@tanlisu: See branch-alias for additional context.
CONTRIBUTING.md
Outdated
Initial versions of new major or minor releases will likely be made on the | ||
master branch directly instead of a maintenance branch. In this case, the | ||
`branch-alias` for `dev-master` in `composer.json` should be updated. For | ||
example, after tagging a new 1.10.0 release on the master branch, | ||
`composer.json` in the master branch may still read: |
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.
Regardless of when releases are tagged, the branch-alias
needs to be changed once we branch off. For example, after branching v1.10
, the branch-alias
for dev-master
should be changed to 1.11.x
.
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.
Good point. I'll move this to a separate section that talks about branching in that case.
8ed2a25
to
69cad71
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.
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.
Thanks!
This was missed after the 1.10.0 release.