----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40310/#review107689 -----------------------------------------------------------
Ship it! Stepping in for Maxim. - Bill Farner On Nov. 16, 2015, 12:11 p.m., John Sirois wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40310/ > ----------------------------------------------------------- > > (Updated Nov. 16, 2015, 12:11 p.m.) > > > Review request for Aurora, Joshua Cohen, Joe Smith, Maxim Khutornenko, and > Zameer Manji. > > > Bugs: AURORA-1532 > https://issues.apache.org/jira/browse/AURORA-1532 > > > Repository: aurora > > > Description > ------- > > This installs the ~newly split off pants python checks contrib plugin. > Release notes for that were here: > https://pypi.python.org/pypi/pantsbuild.pants/0.0.58 > > The plugin provides both python checkstyle (`compile.pythonstyle`), and > a python eval task (`compile.python-eval`). The `python-eval` is turned > off since at least one of the Aurora python targets has files that have > side-effects upon import (a repl is started). > > Now style checks run before compile (and thus before tests) and they > benefit from fingerprinting; ie: if you test your changes, those tests > will run style checks and when you go to commit, those checks will not > be re-run by the commit hook (although files you did not test will still > need to be checked). > > A few production files were fixed up according to style failures coming > from: > + no space after comment opening '#' > + unused variables > + mis-aligned hanging closing parens. > > > Diffs > ----- > > build-support/hooks/pre-commit 619fa9e245be49e4e1f21781c0908cbf744b10ea > build-support/jenkins/build.sh 41a392162f62236771ccbef5c9f94bf84b899f26 > build-support/python/checkstyle 61acc22613acece01580761b25afc7a3edb6b845 > build-support/python/checkstyle-check > b2bfc5dd71193a8056828e9af05a4c16965f32a1 > pants.ini d58908ca8356e53c5317fc8fd7531d5285e21fa7 > src/main/python/apache/aurora/admin/maintenance.py > 6d94c923ae37bf6b827519d3505b100af306296b > src/main/python/apache/aurora/client/api/__init__.py > 6f07a3073a5d422373238619d459fbd09d8adf3d > src/main/python/apache/aurora/client/cli/client.py > 297fb588808c1eebc32ac3374265ba986dab3436 > src/main/python/apache/aurora/client/cli/cron.py > 6376fd014f2a4da29442b5c2c7eb36578b503ba3 > src/main/python/apache/thermos/core/process.py > fe95cb3be01b47616596bd78cb9a919b2e8bd978 > src/main/python/apache/thermos/monitoring/process_collector_psutil.py > f1ec5a9050ac60700c4a8afa905bcf12a9bd8a44 > src/test/python/apache/aurora/admin/test_admin.py > 8e204ab43c6bf69867ea7c32b0a7ba7fb29c0766 > src/test/python/apache/aurora/admin/util.py > 3570407b51613d0a7b4fde8a4794d88b98e150b5 > src/test/python/apache/aurora/client/cli/test_task.py > 5432a3d5f7e150b12bd75db0dac7a9018e1c6636 > > Diff: https://reviews.apache.org/r/40310/diff/ > > > Testing > ------- > > See the discarded https://reviews.apache.org/r/40219/ for the > commit-hook check. This version of that RB engages the same code > and this RB commit was vetted by the same commit-hook. > > > Thanks, > > John Sirois > >