Remove files without an extension from the presubmit checks.

Remove files without an extension from the presubmit checks.

This may include Makefile but it's saner to not include it by default

R=dpranke@chromium.org
BUG=
TEST=updated

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=90403

[Dirk Pranke, 2011-06-24 18:58:34]

LGTM.

http://codereview.chromium.org/7251002/diff/1/presubmit_support.py
File presubmit_support.py (right):

http://codereview.chromium.org/7251002/diff/1/presubmit_support.py#newcode192
presubmit_support.py:192: # Note that ALL CAPS files are black listed in
DEFAULT_BLACK_LIST below.
Aside ... I wonder if we should change this to just be a list of strings to
match on extensions, and transform them into regexps internally. It might be
less flexible, but it would also be clearer and less error-prone.

[M-A Ruel, 2011-06-24 19:11:36]

http://codereview.chromium.org/7251002/diff/1/presubmit_support.py
File presubmit_support.py (right):

http://codereview.chromium.org/7251002/diff/1/presubmit_support.py#newcode192
presubmit_support.py:192: # Note that ALL CAPS files are black listed in
DEFAULT_BLACK_LIST below.
On 2011/06/24 18:58:34, Dirk Pranke wrote:
> Aside ... I wonder if we should change this to just be a list of strings to
> match on extensions, and transform them into regexps internally. It might be
> less flexible, but it would also be clearer and less error-prone.

Make sense but some of them wouldn't work so it would be a mix of raw regex and
list of extensions. I'll do that when I'm bored. :)