Chromium Code Reviews| Index: ios/web/closure_compiler.py |
| diff --git a/ios/web/closure_compiler.py b/ios/web/closure_compiler.py |
| new file mode 100755 |
| index 0000000000000000000000000000000000000000..ca0de686cc1b61c3eff510a82b2a5f392bc197bd |
| --- /dev/null |
| +++ b/ios/web/closure_compiler.py |
| @@ -0,0 +1,10 @@ |
| +#!/usr/bin/python |
| + |
| +import subprocess |
| +import sys |
| + |
| +def __main__(): |
|
Eugene But (OOO till 7-30)
2015/10/08 16:10:41
Optional NIT: s/__main__/main ? Otherwise the name
Eugene But (OOO till 7-30)
2015/10/08 16:10:41
Need 2 blank lines before top level def
Dirk Pranke
2015/10/08 17:49:45
I wouldn't call this change optional. Please chang
Eugene But (OOO till 7-30)
2015/10/08 18:04:11
I use gpylint for this.
Dirk Pranke
2015/10/08 18:22:29
I meant, don't we have some other script that call
sdefresne
2015/10/09 13:54:45
I've looked around but could find nothing. There i
|
| + return subprocess.check_call(["java", "-jar"] + sys.argv[1:]) |
| + |
| +if __name__ == "__main__": |
| + __main__() |