Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(179)

Issue 1905233002: Add framework for basic cr command (Closed)

Created:
4 years, 8 months ago by agable
Modified:
4 years, 7 months ago
Reviewers:
seanmccullough
CC:
chromium-reviews, infra-reviews+infra_chromium.org
Base URL:
https://chromium.googlesource.com/infra/infra.git@master
Target Ref:
refs/heads/master
Project:
infra
Visibility:
Public.

Description

Add framework for basic cr command Establishes a framework for creating subcommands (and adds the necessary packages to the .vendor environment). Adds a stub of the first subcommand, firstrun, which installs the cr system. BUG=605772

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+109 lines, -5 lines) Patch
M go/deps.lock View 5 chunks +11 lines, -5 lines 0 comments Download
M go/deps.yaml View 1 chunk +6 lines, -0 lines 0 comments Download
A go/src/infra/tools/cr/cmd/firstrun.go View 1 chunk +36 lines, -0 lines 0 comments Download
A go/src/infra/tools/cr/cmd/root.go View 1 chunk +40 lines, -0 lines 0 comments Download
A go/src/infra/tools/cr/main.go View 1 chunk +16 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
agable
No application logic here, yet, but felt it made sense to send just the skeleton ...
4 years, 8 months ago (2016-04-21 22:54:40 UTC) #2
seanmccullough
Ok, so thought about this a bit. How about this for a matter of principle: ...
4 years, 7 months ago (2016-04-27 18:17:33 UTC) #3
agable
On 2016/04/27 at 18:17:33, seanmccullough wrote: > Ok, so thought about this a bit. How ...
4 years, 7 months ago (2016-04-27 18:30:58 UTC) #4
agable
4 years, 7 months ago (2016-04-28 22:50:24 UTC) #5
Message was sent while issue was closed.
See https://codereview.chromium.org/1929153002 for a newer version of this CL.

Powered by Google App Engine
This is Rietveld 408576698