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

Issue 553166: Simplify submitting try jobs directly from rietveld. (Closed)

Created:
10 years, 10 months ago by M-A Ruel
Modified:
9 years, 7 months ago
Reviewers:
Mohamed Mansour
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Simplify submitting try jobs directly from rietveld. trychange.py -R <review_url> is sufficient for try jobs when in a checkout directory. TEST=none BUG=none

Patch Set 1 #

Total comments: 1

Patch Set 2 : simplification #

Unified diffs Side-by-side diffs Delta from patch set Stats (+27 lines, -7 lines) Patch
M trychange.py View 1 4 chunks +27 lines, -7 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Mohamed Mansour
lgtm, neato!
10 years, 10 months ago (2010-01-30 05:33:25 UTC) #1
Mohamed Mansour
10 years, 10 months ago (2010-01-30 05:39:44 UTC) #2
Some comment, do what you please.

http://codereview.chromium.org/553166/diff/1/2
File trychange.py (right):

http://codereview.chromium.org/553166/diff/1/2#newcode581
trychange.py:581: options.rietveld_url = re.match(r'^(.*)/\d+$',
Couldn't you just do:
 match = re.match(r'^(.*)/(\d+)$', options.rietveld_url)

options.issue = int(match.group(2))
options.rietveld_url = match.group(1)

Powered by Google App Engine
This is Rietveld 408576698