Do not remove issue from branch after it is landed manually

Do not remove issue from branch after it is landed manually

This is consistent with the way CQ works. Also git-cl-archive won't work if
branch looses info about its matching CL.

R=tandrii@chromium.org

Committed: https://chromium.googlesource.com/chromium/tools/depot_tools/+/2546dcc080884eb817ac0affbd3f16f6c156c2d5

[Sergiy Byelozyorov, 2016-06-29 13:42:47]

The CQ bit was checked by sergiyb@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-29 13:42:54]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tandrii(chromium), 2016-06-29 13:43:46]

LGTM (but git rebase-update would still cleanup this branch, and do other stuff,
that you may or may not want to)

[tandrii(chromium), 2016-06-29 13:45:18]

WARNING: while I think the change is reasonable, you have to on the lookout for
breaking various tooling that relied on this. So, land at your own risk!

[commit-bot: I haz the power, 2016-06-29 13:49:21]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-29 13:49:22]

Dry run: Try jobs failed on following builders:
  Depot Tools Presubmit on master.tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Depot%20Tools%20Presubm...)

[Sergiy Byelozyorov, 2016-06-29 14:13:55]

The CQ bit was checked by sergiyb@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-29 14:14:03]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Sergiy Byelozyorov, 2016-06-29 14:14:34]

On 2016/06/29 13:45:18, tandrii(chromium) wrote:
> WARNING: while I think the change is reasonable, you have to on the lookout
for
> breaking various tooling that relied on this. So, land at your own risk!

I realize that, so if you get any reports - please let me know. I'll also notify
the current trooper.

[commit-bot: I haz the power, 2016-06-29 14:20:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-29 14:20:13]

Dry run: Try jobs failed on following builders:
  Depot Tools Presubmit on master.tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Depot%20Tools%20Presubm...)

[Sergiy Byelozyorov, 2016-06-29 14:21:59]

The CQ bit was checked by sergiyb@chromium.org

[Sergiy Byelozyorov, 2016-06-29 14:22:00]

The patchset sent to the CQ was uploaded after l-g-t-m from tandrii@chromium.org
Link to the patchset: https://codereview.chromium.org/2111543002/#ps40001
(title: "Fix")

[commit-bot: I haz the power, 2016-06-29 14:22:04]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-29 14:26:41]

Description was changed from

==========
Do not remove issue from branch after it is landed manually

This is consistent with the way CQ works. Also git-cl-archive won't work if
branch looses info about its matching CL.

R=tandrii@chromium.org
==========

to

==========
Do not remove issue from branch after it is landed manually

This is consistent with the way CQ works. Also git-cl-archive won't work if
branch looses info about its matching CL.

R=tandrii@chromium.org

Committed:
https://chromium.googlesource.com/chromium/tools/depot_tools/+/2546dcc080884e...
==========

[commit-bot: I haz the power, 2016-06-29 14:26:43]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/tools/depot_tools/+/2546dcc080884e...