Skip to content

Commit

Permalink
Ignore NOTEs when Travis runs make tidy
Browse files Browse the repository at this point in the history
Only print NOTE warnings if the 'TRAVIS' environment variable has not
been set. Addresses #21322.
  • Loading branch information
iKevinY committed Jan 18, 2015
1 parent a833337 commit d1b1b62
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/etc/tidy.py
Expand Up @@ -8,7 +8,7 @@
# option. This file may not be copied, modified, or distributed
# except according to those terms.

import sys, fileinput, subprocess, re
import sys, fileinput, subprocess, re, os
from licenseck import *
import snapshot

Expand Down Expand Up @@ -71,7 +71,7 @@ def do_license_check(name, contents):
if match:
report_err("XXX is no longer necessary, use FIXME")
match = re.match(r'^.*//\s*(NOTE.*)$', line)
if match:
if match and "TRAVIS" not in os.environ:
m = match.group(1)
if "snap" in m.lower():
report_warn(match.group(1))
Expand Down

0 comments on commit d1b1b62

Please sign in to comment.