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

Issue 4310001: Make CheckLicense() to not trigger on empty files. (Closed)

Created:
10 years, 1 month ago by M-A Ruel
Modified:
9 years, 7 months ago
CC:
chromium-reviews, jrbarnette
Visibility:
Public.

Description

Make CheckLicense() to not trigger on empty files. This is especially annoying on empty __init__.py files. TEST=new unit test BUG=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=64904

Patch Set 1 #

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

Messages

Total messages: 3 (0 generated)
M-A Ruel
10 years, 1 month ago (2010-11-02 14:02:31 UTC) #1
jochen (gone - plz use gerrit)
lgtm do you plan to use allow_empty_files somewhere or why did you add it?
10 years, 1 month ago (2010-11-02 18:54:09 UTC) #2
M-A Ruel
10 years, 1 month ago (2010-11-03 13:13:34 UTC) #3
On 2010/11/02 18:54:09, jochen wrote:
> lgtm
> 
> do you plan to use allow_empty_files somewhere or why did you add it?

No, it's just in case someone wants to disable that new functionality.

Powered by Google App Engine
This is Rietveld 408576698