[jira] [Commented] (AURORA-1397) Python style checker should only report errors

2015-07-13 Thread Bill Farner (JIRA) 

[ 
https://issues.apache.org/jira/browse/AURORA-1397?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14625443#comment-14625443
 ] 

Bill Farner commented on AURORA-1397:
-

Upgrading to isort 4.0.0 (latest) gives us this new option, meaning the default 
behavior would be what we're after:
{noformat}
  -vb, --verboseShows verbose output, such as when files are skipped
or when a check is successful.
{noformat}

> Python style checker should only report errors
> --
>
> Key: AURORA-1397
> URL: https://issues.apache.org/jira/browse/AURORA-1397
> Project: Aurora
>  Issue Type: Story
>  Components: Build
>Reporter: Bill Farner
>Assignee: Bill Farner
>Priority: Minor
>
> When running our top-level build script (./build-support/jenkins/build.sh), 
> style check results are reported for python files.  This output is too 
> verbose, as it reports {{SUCCESS}} for all files that pass style check.  
> Change this output to only report errors.
> In particular, this will make for more direct build result messages from 
> build bot.  A counter-example is the current state, as seen in 
> https://reviews.apache.org/r/36392/



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (AURORA-1397) Python style checker should only report errors

2015-07-13 Thread Bill Farner (JIRA) 

[ 
https://issues.apache.org/jira/browse/AURORA-1397?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14625166#comment-14625166
 ] 

Bill Farner commented on AURORA-1397:
-

The noisy command is ./build-support/python/isort-check

> Python style checker should only report errors
> --
>
> Key: AURORA-1397
> URL: https://issues.apache.org/jira/browse/AURORA-1397
> Project: Aurora
>  Issue Type: Story
>  Components: Build
>Reporter: Bill Farner
>Priority: Minor
>
> When running our top-level build script (./build-support/jenkins/build.sh), 
> style check results are reported for python files.  This output is too 
> verbose, as it reports {{SUCCESS}} for all files that pass style check.  
> Change this output to only report errors.
> In particular, this will make for more direct build result messages from 
> build bot.  A counter-example is the current state, as seen in 
> https://reviews.apache.org/r/36392/



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)