[
https://issues.apache.org/jira/browse/STORM-2510?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Erik Weathers updated STORM-2510:
---------------------------------
Description:
Adjust Storm's checkstyle configuration:
* 4-space indent instead of 2-space indent that is default with
google_checks.xml
* exclude the thrift generated code from checkstyle (since we shouldn't be
touching it anyways)
* go with 120 character line-length limits instead of the default of 100 from
google_checks.xml
With those adjustments, the total number of violations will shrink by ~100,000
(~140,000 -> ~40,000).
was:
Adjust Storm's checkstyle configuration:
* 4-space indent instead of 2-space indent that is default with
google_checks.xml
* exclude the thrift generated code from checkstyle (since we shouldn't be
touching it anyways)
* go with 120 character line-length limits instead of the default of 100 from
google_checks.xml
With those adjustments, the total number of violations will shrink by ~75,000.
> adjust checkstyle configurations to decrease violations
> -------------------------------------------------------
>
> Key: STORM-2510
> URL: https://issues.apache.org/jira/browse/STORM-2510
> Project: Apache Storm
> Issue Type: Bug
> Components: build
> Affects Versions: 2.0.0
> Reporter: Erik Weathers
> Assignee: Erik Weathers
>
> Adjust Storm's checkstyle configuration:
> * 4-space indent instead of 2-space indent that is default with
> google_checks.xml
> * exclude the thrift generated code from checkstyle (since we shouldn't be
> touching it anyways)
> * go with 120 character line-length limits instead of the default of 100 from
> google_checks.xml
> With those adjustments, the total number of violations will shrink by
> ~100,000 (~140,000 -> ~40,000).
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
