[jira] [Updated] (PDFBOX-2820) Configure and enable checkstyle for fontbox.

["Glenn Adams (JIRA)" <ji...@apache.org>]

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

Glenn Adams updated PDFBOX-2820:
--------------------------------
    Attachment: fontbox.checkstyle.patch

Patch that configures fontbox validate and site phases to run checkstyle. Updated checkstyle configuration for current version of checkstyle (6.1.1).

> 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: dev-unsubscribe@pdfbox.apache.org
For additional commands, e-mail: dev-help@pdfbox.apache.org