Skip to content

Commit

Permalink
update tidy.py remove collect_file_names:
Browse files Browse the repository at this point in the history
call python os functions inline
  • Loading branch information
jxs committed Aug 24, 2015
1 parent b1f0357 commit 69ddedc
Showing 1 changed file with 4 additions and 14 deletions.
18 changes: 4 additions & 14 deletions python/tidy.py
Expand Up @@ -17,7 +17,7 @@
from licenseck import licenses

filetypes_to_check = [".rs", ".rc", ".cpp", ".c", ".h", ".py", ".toml", ".webidl"]
reftest_directories = ["tests/ref"]
reftest_dir = "./tests/ref"
reftest_filetype = ".list"
python_dependencies = [
"./python/dependencies/flake8-2.4.1-py2.py3-none-any.whl",
Expand Down Expand Up @@ -50,15 +50,6 @@
]


def collect_file_names(top_directories=None):
if top_directories is None:
top_directories = os.listdir(".")
for top_directory in top_directories:
for dirname, dirs, files in os.walk(top_directory):
for basename in files:
yield os.path.join(dirname, basename)


def should_check(file_name):
if ".#" in file_name:
return False
Expand Down Expand Up @@ -393,18 +384,17 @@ def check_reftest_html_files_in_basic_list(reftest_dir):
def scan():
sys.path += python_dependencies

all_files = collect_file_names()
all_files = os.listdir(".")
files_to_check = filter(should_check, all_files)

checking_functions = [check_license, check_by_line, check_flake8, check_toml,
check_rust, check_webidl_spec, check_spec]
errors = collect_errors_for_files(files_to_check, checking_functions)

reftest_files = collect_file_names(reftest_directories)

reftest_files = [os.path.join(reftest_dir, file) for file in os.listdir(reftest_dir)]
reftest_to_check = filter(should_check_reftest, reftest_files)
r_errors = check_reftest_order(reftest_to_check)
not_found_in_basic_list_errors = check_reftest_html_files_in_basic_list(reftest_directories[0])
not_found_in_basic_list_errors = check_reftest_html_files_in_basic_list(reftest_dir)

errors = list(itertools.chain(errors, r_errors, not_found_in_basic_list_errors))

Expand Down

0 comments on commit 69ddedc

Please sign in to comment.