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

Issue 3159010: Don't throw an error if not on stabilizing branch and give more information on exit. (Closed)

Created:
10 years, 4 months ago by sosa
Modified:
9 years, 6 months ago
Reviewers:
petkov
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, anush, sosa
Base URL:
http://src.chromium.org/git/crosutils.git
Visibility:
Public.

Description

Don't throw an error if not on stabilizing branch and give more information on exit. TEST=Ran "cros_mark_as_stable push" with no stabilizing_branch

Patch Set 1 #

Total comments: 1

Patch Set 2 : petkov #

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

Messages

Total messages: 3 (0 generated)
sosa
Should fix errors from cbuildbot
10 years, 4 months ago (2010-08-12 17:24:29 UTC) #1
petkov
LGTM w/ a suggestion. http://codereview.chromium.org/3159010/diff/1/2 File cros_mark_as_stable.py (right): http://codereview.chromium.org/3159010/diff/1/2#newcode129 cros_mark_as_stable.py:129: print 'No work found to ...
10 years, 4 months ago (2010-08-12 17:29:13 UTC) #2
sosa
10 years, 4 months ago (2010-08-12 17:33:13 UTC) #3
good point, will do and commit

Powered by Google App Engine
This is Rietveld 408576698