[jira] [Updated] (FLUME-2937) Integrate checkstyle for non-test classes

2016-06-29 Thread Mike Percy (JIRA)

 [ 
https://issues.apache.org/jira/browse/FLUME-2937?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mike Percy updated FLUME-2937:
--
Summary: Integrate checkstyle for non-test classes  (was: Integrate 
checkstyle)

> Integrate checkstyle for non-test classes
> -
>
> Key: FLUME-2937
> URL: https://issues.apache.org/jira/browse/FLUME-2937
> Project: Flume
>  Issue Type: Improvement
>Reporter: Lior Zeno
>Assignee: Mike Percy
>Priority: Minor
> Fix For: v1.7.0
>
>
> We should add the maven-checkstyle-plugin to the build process. This plugin 
> can fail a build if the code does not honor the style of our project. This 
> way we can make sure that we have one common style in the code. In addition, 
> reviewers can focus on design, correctness, performance and other important 
> coding aspects other than style issues.



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


[jira] [Updated] (FLUME-2937) Integrate checkstyle

2016-06-28 Thread Lior Zeno (JIRA)

 [ 
https://issues.apache.org/jira/browse/FLUME-2937?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Lior Zeno updated FLUME-2937:
-
 Assignee: Mike Percy
Affects Version/s: (was: v1.8.0)
Fix Version/s: (was: v1.8.0)
   v1.7.0

> Integrate checkstyle
> 
>
> Key: FLUME-2937
> URL: https://issues.apache.org/jira/browse/FLUME-2937
> Project: Flume
>  Issue Type: Improvement
>Reporter: Lior Zeno
>Assignee: Mike Percy
>Priority: Minor
> Fix For: v1.7.0
>
>
> We should add the maven-checkstyle-plugin to the build process. This plugin 
> can fail a build if the code does not honor the style of our project. This 
> way we can make sure that we have one common style in the code. In addition, 
> reviewers can focus on design, correctness, performance and other important 
> coding aspects other than style issues.



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