[Release tools] Add instructions how to close the auto-roller.

[Release tools] Add instructions how to close the auto-roller.

R=machenbach@chromium.org
BUG=chromium:494641
LOG=N
NOTRY=true

Committed: https://crrev.com/a023b39f351f1f96a075e63c647a7a40e93ddb5e
Cr-Commit-Position: refs/heads/master@{#29404}

[Michael Hablich, 2015-06-30 14:13:44]

PTAL

[Michael Achenbach, 2015-07-01 05:39:56]

lg

nit: Shorten commit message and make it say what the CL does not what it should
do. E.g.
"Add instructions how to close the auto-roller."

https://codereview.chromium.org/1217173003/diff/1/tools/release/chromium_roll.py
File tools/release/chromium_roll.py (right):

https://codereview.chromium.org/1217173003/diff/1/tools/release/chromium_roll...
tools/release/chromium_roll.py:17:
https://code.google.com/p/v8-wiki/wiki/TriagingIssues/n
Remove /n. In normal strings \n - but in """ strings the actual linebreaks are
used verbatim. Maybe add one more linebreak between the two statements.

[Michael Hablich, 2015-07-01 09:13:31]

https://codereview.chromium.org/1217173003/diff/1/tools/release/chromium_roll.py
File tools/release/chromium_roll.py (right):

https://codereview.chromium.org/1217173003/diff/1/tools/release/chromium_roll...
tools/release/chromium_roll.py:17:
https://code.google.com/p/v8-wiki/wiki/TriagingIssues/n
On 2015/07/01 05:39:56, Michael Achenbach wrote:
> Remove /n. In normal strings \n - but in """ strings the actual linebreaks are
> used verbatim. Maybe add one more linebreak between the two statements.

Done.

[Michael Achenbach, 2015-07-01 09:33:43]

LGTM. Small rietveld gotcha: The CL description exists two times - you changed
the right one (the first line of the description will end up as the commit
message title). But better keep first line CL desc and rietveld title in sync.

[Michael Hablich, 2015-07-01 09:35:35]

The CQ bit was checked by hablich@chromium.org

[commit-bot: I haz the power, 2015-07-01 09:35:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1217173003/20001

[commit-bot: I haz the power, 2015-07-01 09:37:06]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-07-01 09:37:30]

Patchset 2 (id:??) landed as
https://crrev.com/a023b39f351f1f96a075e63c647a7a40e93ddb5e
Cr-Commit-Position: refs/heads/master@{#29404}

[Michael Achenbach, 2015-07-01 09:51:26]

Next gotcha: bugdroid works only cross-repo if the bug is prefixed. E.g.
chromium issues in v8 CLs need BUG=chromium:494641

[Michael Hablich, 2015-07-01 09:54:52]

On 2015/07/01 09:51:26, Michael Achenbach wrote:
> Next gotcha: bugdroid works only cross-repo if the bug is prefixed. E.g.
> chromium issues in v8 CLs need BUG=chromium:494641

Ha, I thought the Chromium repo is the default as Rietveld is correctly
displaying it.