diff options
author | Zuul <zuul@review.openstack.org> | 2018-06-29 04:24:21 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2018-06-29 04:24:21 +0000 |
commit | 31e7e68aa931ffc5999c2415ab26422688010f03 (patch) | |
tree | e39ecb1827bb3fc55b6039912f8dbec484b02c68 /tools | |
parent | 4b08524240141652ef7e14083cbe4a30f3fed8ef (diff) | |
parent | f453a5f64c4534893e64e4a525c466edf01aae96 (diff) | |
download | trove-31e7e68aa931ffc5999c2415ab26422688010f03.tar.gz |
Merge "change pylint wrapper to let messages be prefixes"
Diffstat (limited to 'tools')
-rwxr-xr-x | tools/trove-pylint.py | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/tools/trove-pylint.py b/tools/trove-pylint.py index 785c4bf6..75cea908 100755 --- a/tools/trove-pylint.py +++ b/tools/trove-pylint.py @@ -136,13 +136,14 @@ class Config(object): [filename, codename] in self.config['ignored_file_codes']): return True - fcm_ignore1 = [filename, code, message] - fcm_ignore2 = [filename, codename, message] for fcm in self.config['ignored_file_code_messages']: - if fcm_ignore1 == [fcm[0], fcm[1], fcm[2]]: - return True - - if fcm_ignore2 == [fcm[0], fcm[1], fcm[2]]: + if filename != fcm[0]: + # This ignore rule is for a different file. + continue + if fcm[1] not in (code, codename): + # This ignore rule is for a different code or codename. + continue + if message.startswith(fcm[2]): return True return False |