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

Andreas Lehmkühler commented on PDFBOX-2820:
--------------------------------------------

IMHO we shouldn't maintain one separate checkstyle config for each subproject. 
There is already one in the root dir which should be used for all parts of 
PDFBox

> Configure and enable checkstyle for fontbox.
> --------------------------------------------
>
>                 Key: PDFBOX-2820
>                 URL: https://issues.apache.org/jira/browse/PDFBOX-2820
>             Project: PDFBox
>          Issue Type: Improvement
>          Components: FontBox
>            Reporter: Glenn Adams
>         Attachments: fontbox.checkstyle.patch
>
>
> The fontbox sub-tree contains an apparently old checkstyle configuration 
> file, which no longer works for checkstyle 5 or later. I'm not sure if it is 
> being used or not. In any case, it would be useful (IMO) to update this 
> configuration and enable the use of checkstyle:check in the validate phase 
> and checkstyle:checkstyle in the site phase. The former goal is useful for 
> inducing a fail at build time, while the latter is useful for report 
> generation.
> I will attach a patch to this issue that implements the above. For the time 
> being, I've temporarily disabled a few checks due to their producing a rather 
> large number of warnings. Once this initial patch is merged, I will post 
> another patch that re-enables the checks and fixes existing warnings.
> Note that the current checkstyle plugin (version 2.15) employs checkstyle 
> 6.1.1 by default.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to