diff options
author | Corey Farwell <coreyf@rwell.org> | 2015-07-05 12:10:53 -0700 |
---|---|---|
committer | Corey Farwell <coreyf@rwell.org> | 2015-07-09 19:42:31 +0900 |
commit | 0ec2375cabb69abbfd1014bfb68136fdea6edadb (patch) | |
tree | 8a24a61ae92605810bdd7163aad10204833e9a99 /components/script/dom/bindings/codegen/pythonpath.py | |
parent | fe17067d6a30b85a0346fd1ccb2b95e2081e6962 (diff) | |
download | servo-0ec2375cabb69abbfd1014bfb68136fdea6edadb.tar.gz servo-0ec2375cabb69abbfd1014bfb68136fdea6edadb.zip |
Remove tidy blacklist for 'script/dom/bindings/*'
Recently, I found myself reading through the Python codegen scripts that
live in 'components/script/dom/bindings/*' and noticed that there were
many tidy violations: unnecessary semicolons, weird spacing, unused
variables, lack of license headers, etc. Considering these files are now
living in our tree and mostly maintained directly by contributors of
Servo (as opposed to being from upstream), I feel these files should not
be excluded from our normal tidy process. This commit removes the
blacklist on these files and fixes all tidy violations.
I added these subdirectories to the blacklist because they appear to be
maintained upstream somewhere else:
* "components/script/dom/bindings/codegen/parser/*",
* "components/script/dom/bindings/codegen/ply/*",
Also, I added a '# noqa' comment which tells us to ignore the
flake8 errors for that line. I chose to ignore this (instead of fixing
it) to make the work for this commit simpler for me.
Diffstat (limited to 'components/script/dom/bindings/codegen/pythonpath.py')
-rw-r--r-- | components/script/dom/bindings/codegen/pythonpath.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/components/script/dom/bindings/codegen/pythonpath.py b/components/script/dom/bindings/codegen/pythonpath.py index 49b2d2f740f..793089551b5 100644 --- a/components/script/dom/bindings/codegen/pythonpath.py +++ b/components/script/dom/bindings/codegen/pythonpath.py @@ -54,7 +54,8 @@ def main(args): # Freeze scope here ... why this makes things work I have no idea ... frozenglobals = globals() -import sys, os +import sys +import os if __name__ == '__main__': main(sys.argv[1:]) |