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

Issue 6820076: Add author to Change object. Don't set it yet. (Closed)

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

Description

Add author to Change object. Don't set it yet. It's mostly to be used with the commit queue. R=dpranke@chromium.org BUG= TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=81326

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+134 lines, -59 lines) Patch
M gcl.py View 1 chunk +2 lines, -1 line 0 comments Download
M git_cl.py View 1 chunk +8 lines, -2 lines 0 comments Download
M presubmit_support.py View 5 chunks +7 lines, -2 lines 0 comments Download
M tests/presubmit_unittest.py View 27 chunks +117 lines, -54 lines 3 comments Download

Messages

Total messages: 3 (0 generated)
M-A Ruel
9 years, 8 months ago (2011-04-12 20:26:27 UTC) #1
Dirk Pranke
LGTM. Up to you if you want to make the suggested changes. http://codereview.chromium.org/6820076/diff/1/tests/presubmit_unittest.py File tests/presubmit_unittest.py ...
9 years, 8 months ago (2011-04-12 20:49:24 UTC) #2
M-A Ruel
9 years, 8 months ago (2011-04-12 23:13:24 UTC) #3
http://codereview.chromium.org/6820076/diff/1/tests/presubmit_unittest.py
File tests/presubmit_unittest.py (right):

http://codereview.chromium.org/6820076/diff/1/tests/presubmit_unittest.py#new...
tests/presubmit_unittest.py:351: change = presubmit.Change(
On 2011/04/12 20:49:24, Dirk Pranke wrote:
> maybe you should pull this into a helper routine (e.g., get_change()) with
> optional params to specify which parameters you care about?

I thought about that but didn't feel like doing it this time. Maybe next time.
:)

Powered by Google App Engine
This is Rietveld 408576698