aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From John Sirois <jsir...@apache.org>
Subject Re: Review Request 51484: Re-enable python style check in the integration build.
Date Mon, 29 Aug 2016 16:44:15 GMT

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


Ship it!




Ship It!

- John Sirois


On Aug. 27, 2016, 10:40 p.m., Santhosh Kumar Shanmugham wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51484/
> -----------------------------------------------------------
> 
> (Updated Aug. 27, 2016, 10:40 p.m.)
> 
> 
> Review request for Aurora, John Sirois and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Re-enable python style check in the integration build.
> 
> pants test does not appear to invoke the python checkstyle. Re-enable
> it by explicitly calling in the integration build script. Also fix the
> few issues that have already been commited.
> 
> 
> Diffs
> -----
> 
>   build-support/jenkins/build.sh 1de144628f0e687eb3a191b0aa819bae018a8f37 
>   src/main/python/apache/aurora/client/cli/context.py f1a256a8d09d23d8d4d4ee7d264be0fe376398c4

>   src/main/python/apache/aurora/client/cli/update.py 23aaa2c1b67599420408633733e4581553f7151b

>   src/main/python/apache/thermos/core/process.py 78e7d788f14a1611031e3c6e255c77768daabac4

>   src/test/python/apache/aurora/executor/common/test_sandbox.py b30a5bc733ac5ccace0b93b68e2ee6f04ce31184

>   src/test/python/apache/thermos/core/test_process.py 49f52d947471c8ea8daae1708f4d26afa4c6ee0a

> 
> Diff: https://reviews.apache.org/r/51484/diff/
> 
> 
> Testing
> -------
> 
> ./build-support/jenkins/build.sh
> 
> 
> Thanks,
> 
> Santhosh Kumar Shanmugham
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message