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