You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pig.apache.org by "Gianmarco De Francisci Morales (JIRA)" <ji...@apache.org> on 2012/10/29 03:35:12 UTC

[jira] [Updated] (PIG-3008) Fix whitespace in Pig code

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

Gianmarco De Francisci Morales updated PIG-3008:
------------------------------------------------

    Attachment: checkstyle.xml

Agreed, let's start with this.
Here a first stab for a checkstyle with FileTabCharacter, Indentation, and RegexpSingleLine for trailing white space.
I threw in a few other things which I think are useful as well:
Checking all files have Apache header, checking imports and using ==/!+ with Strings.
                
> 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