This is an archive of the discontinued LLVM Phabricator instance.

docs: Add instructions for how to submit a merge request
ClosedPublic

Authored by tstellar on Sep 15 2017, 2:53 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

tstellar created this revision.Sep 15 2017, 2:53 PM
hans added a subscriber: hans.Sep 15 2017, 3:03 PM

Good stuff!

docs/HowToReleaseLLVM.rst
266 ↗(On Diff #115505)

Might as well make bugs.llvm.org a link I suppose?

267 ↗(On Diff #115505)

s/$PROJ/llvm/ ?

Maybe expand the example a little in case people are confused about X and Y, e.g.

To request a patch to be merged to the branch for the 5.0.1 stable release, use this...

269 ↗(On Diff #115505)

Link again, and maybe mention marking it as a blocker of the current release bug.. which I suppose one would have to search for, or know from the mailing list.

tstellar updated this revision to Diff 115530.Sep 15 2017, 10:29 PM

Address review comments.

hans accepted this revision.Sep 18 2017, 11:57 AM

Very nice, thanks!

This revision is now accepted and ready to land.Sep 18 2017, 11:57 AM
This revision was automatically updated to reflect the committed changes.