-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[Doc] Update the Branches doc with LLVM Project info #27859
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
docs/Branches.md
Outdated
| ------ | ------ | ------ | ||
| master | stable | stable | ||
| Swift | LLVM Project | | ||
| ------ | ------ | |
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.
Nitpick: the dashes should extend under the whole title.
docs/Branches.md
Outdated
- LLVM/Clang/compiler-rt: new commits go to `upstream-with-swift` | ||
|
||
- LLDB: new commits go to `stable` | ||
- LLVM Project: new commits go to `swift/master-next` | ||
|
||
...then cherry-pick to the release branch (`swift-x.y-branch`) if necessary, following the appropriate release process. (Usually this means filling out a standard template, finding someone to review your code if that hasn't already happened, and getting approval from that repo's *release manager.)* |
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.
This should probably say "swift-x.y-branch
or swift/swift-x.y-branch
".
docs/Branches.md
Outdated
- `stable` is automerged into `upstream-with-swift` | ||
- llvm.org's `master` is *also* automerged into `upstream-with-swift` | ||
### LLVM Project | ||
- `swift-x.y-branch` (the *latest* release branch) is automerged into `swift/master` |
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.
This should say swift/swift-x.y.branch
(here and below).
@swift-ci smoke test |
docs/Branches.md
Outdated
@@ -68,6 +68,6 @@ Some branches are *automerged* into other branches, to keep them in sync. This i | |||
- `master` is automerged into `master-next` | |||
|
|||
### LLVM Project | |||
- `swift-x.y-branch` (the *latest* release branch) is automerged into `swift/master` | |||
- `swift/swift-x.y-branch` (the *latest* release branch) is automerged into `swift/master` | |||
- llvm.org's `master` is automerged into `swift/master-next` | |||
- llvm.org's release branch *may* be automerged into `swift-x.y-branch`, if they are in sync |
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.
Here too, sorry.
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.
Fixed, thanks!
No description provided.