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

Issue 6628072: fix typos in CheckOwners() (Closed)

Created:
9 years, 9 months ago by Dirk Pranke
Modified:
9 years, 7 months ago
Reviewers:
M-A Ruel
CC:
chromium-reviews, M-A Ruel
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 1

Patch Set 2 : update w/ feedback from maruel #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -8 lines) Patch
M presubmit_canned_checks.py View 1 1 chunk +8 lines, -8 lines 1 comment Download

Messages

Total messages: 4 (0 generated)
Dirk Pranke
9 years, 9 months ago (2011-03-08 02:14:10 UTC) #1
M-A Ruel
lgtm http://codereview.chromium.org/6628072/diff/1/presubmit_canned_checks.py File presubmit_canned_checks.py (right): http://codereview.chromium.org/6628072/diff/1/presubmit_canned_checks.py#newcode630 presubmit_canned_checks.py:630: affected_files = set([str(f) for f in I usually ...
9 years, 9 months ago (2011-03-08 19:06:21 UTC) #2
Dirk Pranke
On 2011/03/08 19:06:21, Marc-Antoine Ruel wrote: > lgtm > > http://codereview.chromium.org/6628072/diff/1/presubmit_canned_checks.py > File presubmit_canned_checks.py (right): ...
9 years, 9 months ago (2011-03-08 22:28:22 UTC) #3
M-A Ruel
9 years, 9 months ago (2011-03-08 22:42:16 UTC) #4
still lgtm

http://codereview.chromium.org/6628072/diff/7001/presubmit_canned_checks.py
File presubmit_canned_checks.py (right):

http://codereview.chromium.org/6628072/diff/7001/presubmit_canned_checks.py#n...
presubmit_canned_checks.py:630: affected_files = set([f.LocalPath() for f in
I don't think the [] are necessary. You can remove if not.

Powered by Google App Engine
This is Rietveld 408576698