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

Gianmarco De Francisci Morales commented on PIG-3008:
-----------------------------------------------------

1. I am assuming we will integrate this with Ant. The checkstyle ant target 
wants a list of file to run on. So if you are able to identify the list of 
files you changed automatically, then yes. How do you define changed?
2. Yes, as long as there is a compatible formatter profile (i.e. it is not too 
complex). For our use case it should be fine. I know it can be done with 
Eclipse. For other IDEs I guess it can be done as well but don't know how.

We also need integration with Jenkins for automatic builds.
                
> Fix whitespace in Pig code
> --------------------------
>
>                 Key: PIG-3008
>                 URL: https://issues.apache.org/jira/browse/PIG-3008
>             Project: Pig
>          Issue Type: Improvement
>            Reporter: Jonathan Coveney
>             Fix For: 0.12
>
>         Attachments: checkstyle.xml
>
>
> This JIRA exists mainly to get a conversation started. We've talked about it 
> before, and it's a tricky issue. That said, some of the Pig code is super, 
> super gnarly. We need some sort of path that will let it eventually be 
> fix-able.
> I posit: any file that hasn't been touched for over 6 months is eligible for 
> a whitespace patch.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to