Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-01 03:30:33 +01:00
Code Issues Projects Releases Packages Wiki Activity

Auto merge of #6274 - frewsxcv:tidy-all-the-things, r=metajack

Browse source 
After #6268 merged, I found out that everything in the tree currently
passes the tidy checker.

---------------

It is worth pointing out that I personally do not know if this is a feature wanted by others. Once I noticed everything was already tidy, I decided to open this PR to start the conversation (if any). I am not convinced this is really a Good Thing, so my feelings would not get hurt if this was closed immediately.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6274)
<!-- Reviewable:end -->
This commit is contained in:
bors-servo 2015-06-03 11:12:34 -05:00
commit 055182dae1
1 changed files with 7 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

10
python/tidy.py
View file

@ -15,7 +15,6 @@ import itertools
import re import re
from licenseck import licenses from licenseck import licenses
directories_to_check = ["ports", "components", "tests", "python"]
filetypes_to_check = [".rs", ".rc", ".cpp", ".c", ".h", ".py"] filetypes_to_check = [".rs", ".rc", ".cpp", ".c", ".h", ".py"]
reftest_directories = ["tests/ref"] reftest_directories = ["tests/ref"]
reftest_filetype = ".list" reftest_filetype = ".list"
@ -40,10 +39,15 @@ ignored_files = [
# MIT license # MIT license
"components/util/deque/mod.rs", "components/util/deque/mod.rs",
# Hidden files/directories
".*",
] ]
def collect_file_names(top_directories): def collect_file_names(top_directories=None):
if top_directories is None:
top_directories = os.listdir(".")
for top_directory in top_directories: for top_directory in top_directories:
for dirname, dirs, files in os.walk(top_directory): for dirname, dirs, files in os.walk(top_directory):
for basename in files: for basename in files:
@ -142,7 +146,7 @@ def get_reftest_names(line):
def scan(): def scan():
all_files = collect_file_names(directories_to_check) all_files = collect_file_names()
files_to_check = filter(should_check, all_files) files_to_check = filter(should_check, all_files)
checking_functions = [check_license, check_by_line] checking_functions = [check_license, check_by_line]