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

Issue 4218006: Add rietveld member to ChangeInfo and use this value to contact rietveld. (Closed)

Created:
10 years, 1 month ago by M-A Ruel
Modified:
9 years, 7 months ago
Reviewers:
jam
CC:
chromium-reviews, M-A Ruel, jochen (gone - plz use gerrit)
Visibility:
Public.

Description

Add rietveld member to ChangeInfo and use this value to contact rietveld. This makes gcl to use a specific rietveld instance per change list. This is useful when moving from one rietveld instance to another, so in-flight reviews are sent to the right instance. Also clear gcl.CODEREVIEW_SETTINGS between gcl_unittest tests to remove tests interference. TEST=updated unit tests BUG=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=64631

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+96 lines, -89 lines) Patch
M gcl.py View 14 chunks +62 lines, -65 lines 0 comments Download
M tests/gcl_unittest.py View 13 chunks +34 lines, -24 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
M-A Ruel
10 years, 1 month ago (2010-11-01 15:10:44 UTC) #1
M-A Ruel
Forwarding to John since I think Jochen left for the day.
10 years, 1 month ago (2010-11-01 17:15:28 UTC) #2
jam
10 years, 1 month ago (2010-11-01 17:48:42 UTC) #3
lgtm

Powered by Google App Engine
This is Rietveld 408576698