Move gitiles module from milo to common/api.

Move gitiles module from milo to common/api.

I'll be improving it and making use of it in scheduler.

R=iannucci@chromium.org

Bug: 646067
Review-Url: https://codereview.chromium.org/2977083002
Committed: https://github.com/luci/luci-go/commit/915ce57f5cd2262f6324a5af4bf7643f566e1dea

[tandrii(chromium), 2017-07-13 20:49:22]

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

[commit-bot: I haz the power, 2017-07-13 20:49:27]

Dry run: CQ is trying da patch.

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

[tandrii(chromium), 2017-07-13 20:51:21]

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

[commit-bot: I haz the power, 2017-07-13 20:51:27]

Dry run: CQ is trying da patch.

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

[tandrii(chromium), 2017-07-13 20:51:38]

ptal

https://codereview.chromium.org/2977083002/diff/1/common/api/gitiles/gitiles.go
File common/api/gitiles/gitiles.go (left):

https://codereview.chromium.org/2977083002/diff/1/common/api/gitiles/gitiles....
common/api/gitiles/gitiles.go:30: type Repo struct {
this was unused, and I don't like Repo being pinned to Branch.

[commit-bot: I haz the power, 2017-07-13 20:58:17]

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

[commit-bot: I haz the power, 2017-07-13 20:58:19]

Dry run: This issue passed the CQ dry run.

[iannucci, 2017-07-13 21:23:27]

lgtm

https://codereview.chromium.org/2977083002/diff/20001/milo/frontend/console/c...
File milo/frontend/console/console.go (right):

https://codereview.chromium.org/2977083002/diff/20001/milo/frontend/console/c...
milo/frontend/console/console.go:129: // Move things into our own datastructure.
comment not needed

[tandrii(chromium), 2017-07-14 10:13:58]

https://codereview.chromium.org/2977083002/diff/20001/milo/frontend/console/c...
File milo/frontend/console/console.go (right):

https://codereview.chromium.org/2977083002/diff/20001/milo/frontend/console/c...
milo/frontend/console/console.go:129: // Move things into our own datastructure.
On 2017/07/13 21:23:27, iannucci wrote:
> comment not needed

Done.

[tandrii(chromium), 2017-07-14 10:14:10]

The CQ bit was checked by tandrii@chromium.org

[tandrii(chromium), 2017-07-14 10:14:10]

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/2977083002/#ps40001
(title: "-comment")

[commit-bot: I haz the power, 2017-07-14 10:14:18]

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-07-14 10:19:51]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1500027250050750,
"parent_rev": "b783a30c7cfed7b42d232fd37ce1f8950d6e731c", "commit_rev":
"915ce57f5cd2262f6324a5af4bf7643f566e1dea"}

[commit-bot: I haz the power, 2017-07-14 10:19:55]

Description was changed from

==========
Move gitiles module from milo to common/api.

I'll be improving it and making use of it in scheduler.

R=iannucci@chromium.org

Bug: 646067
==========

to

==========
Move gitiles module from milo to common/api.

I'll be improving it and making use of it in scheduler.

R=iannucci@chromium.org

Bug: 646067
Review-Url: https://codereview.chromium.org/2977083002
Committed:
https://github.com/luci/luci-go/commit/915ce57f5cd2262f6324a5af4bf7643f566e1dea
==========

[commit-bot: I haz the power, 2017-07-14 10:19:56]

Committed patchset #3 (id:40001) as
https://github.com/luci/luci-go/commit/915ce57f5cd2262f6324a5af4bf7643f566e1dea