> On Oct. 29, 2015, 4:15 p.m., Zameer Manji wrote:
> > John, would you mind filing a ticket to track the followup work to use the 
> > built in pants checkstyle?
> 
> Joe Smith wrote:
>     https://issues.apache.org/jira/browse/AURORA-1532

Thanks Joe.


- John


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39784/#review104483
-----------------------------------------------------------


On Oct. 29, 2015, 4:05 p.m., John Sirois wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39784/
> -----------------------------------------------------------
> 
> (Updated Oct. 29, 2015, 4:05 p.m.)
> 
> 
> Review request for Aurora, Joe Smith, Bill Farner, and Zameer Manji.
> 
> 
> Bugs: AURORA-1499
>     https://issues.apache.org/jira/browse/AURORA-1499
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> This also brings Aurora up to pex 1.1.0 and switches to the pants
> setup script; an equivalent to gradlew.  Of note, the script is checked
> in with chmod 555, its not intended to be edited.
> 
> Although pants now includes checkstyle built in, conversion to use it is
> left for follow-on work.
> 
> Also adapt make-pycharm-virtualenv which previously relied on the local
> bootstrapping of pants - now hidden away by the pants setup script.
> 
>  .pantsversion                                  |   1 -
>  3rdparty/python/requirements.txt               |   2 +-
>  BUILD                                          |  25 -------------
>  BUILD.tools                                    |  19 ++++++++--
>  build-support/jenkins/build.sh                 |   2 +-
>  build-support/pants_requirements.txt           |  30 ---------------
>  build-support/python/make-pycharm-virtualenv   |  11 +++++-
>  build-support/python/update-pants-requirements |  34 -----------------
>  pants                                          | 102 
> +++++++++++++++++++++++++++++++++++++++-----------
>  pants.ini                                      |  63 
> ++-----------------------------
>  src/main/python/apache/aurora/tools/BUILD      |   2 +-
>  src/main/python/apache/thermos/observer/BUILD  |   2 +-
>  12 files changed, 114 insertions(+), 179 deletions(-)
> 
> 
> Diffs
> -----
> 
>   .pantsversion 78bae5bb6d254d014e35be0b828497f1509d80bd 
>   3rdparty/python/requirements.txt 1eeb36dee1a0ebd33999bbb3327338a51cba00d7 
>   BUILD 7de0c74b03ba609576867ba96885858c0908f2e9 
>   BUILD.tools 75698a5ded7914a4d22ab7ae769d9ed1576531e4 
>   build-support/jenkins/build.sh 5606bb157cb117a588f363382d7c8841ae957138 
>   build-support/pants_requirements.txt 
> fad8da5ce4c03f25554394bc628d4fbb0fe6cd69 
>   build-support/python/make-pycharm-virtualenv 
> d7bd41835bcd9a8fe62ea522a177bfa7830a897b 
>   build-support/python/update-pants-requirements 
> 82f7c5136fad52f4bc2a458beb5f34f0cc7f6cec 
>   pants 6f3526ef76fc37e3215b0673bcb1725d205a1c95 
>   pants.ini dd4ba668586ee5bd1c888f315429e661adb6a480 
>   src/main/python/apache/aurora/tools/BUILD 
> e5ac75838cbe98bbef4e35f6f300b6a6df5e7de5 
>   src/main/python/apache/thermos/observer/BUILD 
> d7eedabc0930711530b45ac98a1159e69d1a0c00 
> 
> Diff: https://reviews.apache.org/r/39784/diff/
> 
> 
> Testing
> -------
> 
> Locally: `./pants test.pytest --no-fast src/test/python:: -- -v`
> 
> Also generated a pycharm project via:
>   `./build-support/python/make-pycharm-virtualenv`
> Confirmed library source linking worked as did running unit tests
> via the IDE.
> 
> Also grepped for pants commands in the repo, found `binary` and `setup-py`
> and confirmed these worked.
> 
> 
> Thanks,
> 
> John Sirois
> 
>

Reply via email to