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

Issue 113294: Add presubmit checks to look for BUG= and TEST= tags in change list descripti... (Closed)

Created:
11 years, 7 months ago by kuchhal
Modified:
9 years, 7 months ago
Reviewers:
M-A Ruel
CC:
chromium-reviews_googlegroups.com, Mark Larson
Visibility:
Public.

Description

Add presubmit checks to look for BUG= and TEST= tags in change list description (Part 1 of 2). BUG=11287 TEST=gcl commit should fail unless BUG= & TEST= exist in change list description or --no_presubmit is used. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=15905

Patch Set 1 #

Total comments: 1

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+49 lines, -3 lines) Patch
M presubmit.py View 1 1 chunk +2 lines, -1 line 0 comments Download
M presubmit_canned_checks.py View 1 1 chunk +18 lines, -0 lines 0 comments Download
M tests/presubmit_unittest.py View 1 1 chunk +29 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
kuchhal
11 years, 7 months ago (2009-05-12 20:19:49 UTC) #1
M-A Ruel
11 years, 7 months ago (2009-05-12 20:23:44 UTC) #2
lgtm even though I think it'd be better to have these be warnings.

http://codereview.chromium.org/113294/diff/1/2
File presubmit_canned_checks.py (right):

http://codereview.chromium.org/113294/diff/1/2#newcode23
Line 23: return [output_api.PresubmitError(
Shouldn't it be a warning?

Powered by Google App Engine
This is Rietveld 408576698