[recipes.py] move refs arg parsing to its module

[recipes.py] move refs arg parsing to its module

R=dnj@chromium.org, phajdan.jr@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2845843002
Committed: https://github.com/luci/recipes-py/commit/bcf9286338730769356d98e274b71e1b6418f487

[iannucci, 2017-04-27 00:44:39]

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

[commit-bot: I haz the power, 2017-04-27 00:44:50]

Dry run: 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-04-27 00:47:20]

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

[commit-bot: I haz the power, 2017-04-27 00:47:21]

Dry run: Try jobs failed on following builders:
  Recipes-py Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/35c5afd4ac4ece10)

[iannucci, 2017-04-27 02:39:06]

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

[commit-bot: I haz the power, 2017-04-27 02:39:11]

Dry run: 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-04-27 02:46:03]

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

[commit-bot: I haz the power, 2017-04-27 02:46:04]

Dry run: Try jobs failed on following builders:
  Recipes-py Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/35c6189d89f9b210)

[iannucci, 2017-04-27 04:21:35]

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

[commit-bot: I haz the power, 2017-04-27 04:21:46]

Dry run: 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-04-27 04:26:16]

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

[commit-bot: I haz the power, 2017-04-27 04:26:17]

Dry run: This issue passed the CQ dry run.

[dnj, 2017-04-27 16:40:40]

lgtm

https://codereview.chromium.org/2845843002/diff/40001/recipe_engine/refs.py
File recipe_engine/refs.py (right):

https://codereview.chromium.org/2845843002/diff/40001/recipe_engine/refs.py#n...
recipe_engine/refs.py:24: modules = args.modules
nit: in other CLs, you would have used "args.modules" instead of creating a
local version. Maybe do this consistently?

[iannucci, 2017-04-29 07:20:17]

The CQ bit was checked by iannucci@chromium.org

[commit-bot: I haz the power, 2017-04-29 07:20:23]

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-04-29 07:23:13]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1493450417405800,
"parent_rev": "0b1247f8506739b0c6d6adc1f8145cb4d0d3cda8", "commit_rev":
"bcf9286338730769356d98e274b71e1b6418f487"}

[commit-bot: I haz the power, 2017-04-29 07:23:16]

Description was changed from

==========
[recipes.py] move refs arg parsing to its module

R=dnj@chromium.org, phajdan.jr@chromium.org
BUG=
==========

to

==========
[recipes.py] move refs arg parsing to its module

R=dnj@chromium.org, phajdan.jr@chromium.org
BUG=

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

[commit-bot: I haz the power, 2017-04-29 07:23:17]

Committed patchset #3 (id:40001) as
https://github.com/luci/recipes-py/commit/bcf9286338730769356d98e274b71e1b641...