test: promote --train to its own subcommand

test: promote --train to its own subcommand

It turns out it's more discoverable that way, and the switch was confusing for people.

BUG=chromium:699120
Review-Url: https://codereview.chromium.org/2790493007
Committed: https://github.com/luci/recipes-py/commit/e02889f17c4ac35b7fb99915c5e05a9dde1347c0

[Paweł Hajdan Jr., 2017-03-31 17:43:42]

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

[commit-bot: I haz the power, 2017-03-31 17:43:50]

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

[Paweł Hajdan Jr., 2017-03-31 17:44:07]

Description was changed from

==========
test: promote --train to its own subcommand

BUG=chromium:699120
==========

to

==========
test: promote --train to its own subcommand

It turns out it's more discoverable that way, and the switch was confusing for
people.

BUG=chromium:699120
==========

[commit-bot: I haz the power, 2017-03-31 17:47:40]

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

[commit-bot: I haz the power, 2017-03-31 17:47:41]

Dry run: This issue passed the CQ dry run.

[Paweł Hajdan Jr., 2017-03-31 17:48:30]

phajdan.jr@chromium.org changed reviewers:
+ iannucci@chromium.org

[iannucci, 2017-03-31 17:50:09]

lgtm

https://codereview.chromium.org/2790493007/diff/1/recipe_engine/test.py
File recipe_engine/test.py (right):

https://codereview.chromium.org/2790493007/diff/1/recipe_engine/test.py#newco...
recipe_engine/test.py:738: help='re-generate recipe expectations')
let's add a deprecation text in the help too.

[Paweł Hajdan Jr., 2017-03-31 17:51:44]

https://codereview.chromium.org/2790493007/diff/1/recipe_engine/test.py
File recipe_engine/test.py (right):

https://codereview.chromium.org/2790493007/diff/1/recipe_engine/test.py#newco...
recipe_engine/test.py:738: help='re-generate recipe expectations')
On 2017/03/31 17:50:09, iannucci wrote:
> let's add a deprecation text in the help too.

Done.

[Paweł Hajdan Jr., 2017-03-31 17:51:47]

The CQ bit was checked by phajdan.jr@chromium.org

[Paweł Hajdan Jr., 2017-03-31 17:51:48]

The patchset sent to the CQ was uploaded after l-g-t-m from
iannucci@chromium.org
Link to the patchset: https://codereview.chromium.org/2790493007/#ps20001
(title: "deprectaed")

[commit-bot: I haz the power, 2017-03-31 17:51:51]

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, 2017-03-31 17:55:03]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490982707831340,
"parent_rev": "09292e372498915b899f480ce9f384057dc5710c", "commit_rev":
"e02889f17c4ac35b7fb99915c5e05a9dde1347c0"}

[commit-bot: I haz the power, 2017-03-31 17:55:08]

Description was changed from

==========
test: promote --train to its own subcommand

It turns out it's more discoverable that way, and the switch was confusing for
people.

BUG=chromium:699120
==========

to

==========
test: promote --train to its own subcommand

It turns out it's more discoverable that way, and the switch was confusing for
people.

BUG=chromium:699120

Review-Url: https://codereview.chromium.org/2790493007
Committed:
https://github.com/luci/recipes-py/commit/e02889f17c4ac35b7fb99915c5e05a9dde1...
==========

[commit-bot: I haz the power, 2017-03-31 17:55:09]

Committed patchset #2 (id:20001) as
https://github.com/luci/recipes-py/commit/e02889f17c4ac35b7fb99915c5e05a9dde1...