Check all files by default

Have an explicit list of exemptions for specific checks rather than
whitelisting files to check. Some checks, such as permissions, should
apply to all files.

Signed-off-by: Gilles Peskine <Gilles.Peskine@arm.com>
This commit is contained in:
Gilles Peskine 2020-05-10 16:54:10 +02:00
parent e856ba11a1
commit 125f89b893

View File

@ -222,23 +222,6 @@ class IntegrityChecker:
self.check_repo_path() self.check_repo_path()
self.logger = None self.logger = None
self.setup_logger(log_file) self.setup_logger(log_file)
self.extensions_to_check = (
".bat",
".c",
".data",
".dsp",
".function",
".h",
".md",
".pl",
".py",
".sh",
".sln",
".vcxproj",
"/CMakeLists.txt",
"/ChangeLog",
"/Makefile",
)
self.excluded_directories = [ self.excluded_directories = [
'.git', '.git',
'mbed-os', 'mbed-os',
@ -285,8 +268,6 @@ class IntegrityChecker:
dirs[:] = sorted(d for d in dirs if not self.prune_branch(root, d)) dirs[:] = sorted(d for d in dirs if not self.prune_branch(root, d))
for filename in sorted(files): for filename in sorted(files):
filepath = os.path.join(root, filename) filepath = os.path.join(root, filename)
if not filepath.endswith(self.extensions_to_check):
continue
for issue_to_check in self.issues_to_check: for issue_to_check in self.issues_to_check:
if issue_to_check.should_check_file(filepath): if issue_to_check.should_check_file(filepath):
issue_to_check.check_file_for_issue(filepath) issue_to_check.check_file_for_issue(filepath)