Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(161)

Issue 171450043: Fix example in help text of batcharchive command. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 3 months ago by Andrew Wang
Modified:
11 years, 3 months ago
Reviewers:
vadimsh, M-A
CC:
swarming-eng_googlegroups.com, csharp+cc_chromium.org, vadimsh+cc_chromium.org
Base URL:
https://code.google.com/p/swarming.client/@master
Visibility:
Public.

Description

Fix example in help text of batcharchive command. BUG= R=vadimsh@chromium.org Committed: https://code.google.com/p/swarming/source/detail?repo=client&r=83648557ff4ef8c87e4d43a8114d2408561e9255

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -3 lines) Patch
M isolate.py View 1 chunk +3 lines, -3 lines 0 comments Download

Messages

Total messages: 5
vadimsh
LGTM I've added you to committers list for now. You should be able to use ...
11 years, 3 months ago (2014-11-11 02:04:35 UTC) #1
vadimsh
On 2014/11/11 02:04:35, vadimsh wrote: > LGTM > > I've added you to committers list ...
11 years, 3 months ago (2014-11-11 02:14:51 UTC) #2
M-A
On 2014/11/11 02:14:51, vadimsh wrote: > On 2014/11/11 02:04:35, vadimsh wrote: > > LGTM > ...
11 years, 3 months ago (2014-11-14 21:00:49 UTC) #3
Andrew Wang
Committed patchset #1 (id:1) manually as 83648557ff4ef8c87e4d43a8114d2408561e9255 (presubmit successful).
11 years, 3 months ago (2014-11-14 21:28:17 UTC) #4
Andrew Wang
11 years, 3 months ago (2014-11-14 21:28:54 UTC) #5
Message was sent while issue was closed.
On 2014/11/14 21:00:49, M-A wrote:
> On 2014/11/11 02:14:51, vadimsh wrote:
> > On 2014/11/11 02:04:35, vadimsh wrote:
> > > LGTM
> > > 
> > > I've added you to committers list for now. You should be able to use 'git
cl
> > > land' to commit this (assuming you've configured your .netrc for
> authenticated
> > > access as instructed here https://code.google.com/hosting/settings)
> > 
> > Removed you from committers list for now :) See the email about CLA stuff.
> 
> Added back, you are free to commit.

Thanks Vadim & M-A!
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b