You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@tika.apache.org by "Konstantin Gribov (Jira)" <ji...@apache.org> on 2022/11/19 21:04:00 UTC

[jira] [Commented] (TIKA-3324) Add checkstyle checker

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

Konstantin Gribov commented on TIKA-3324:
-----------------------------------------

I certainly lost against checkstyle plugin. When I just run {{mvn checkstyle:checkstyle}} it fails on {{tika-core}} with something like 5.7k errors.

What do you think about using [spotless|https://github.com/diffplug/spotless]? It supports [ratchet|https://github.com/diffplug/spotless/tree/main/plugin-gradle#ratchet] mode to avoid reformatting all files at once and to force reformat only on changed files. I'm going to experiment with it in a separate branch for POMs at first.

> Add checkstyle checker
> ----------------------
>
>                 Key: TIKA-3324
>                 URL: https://issues.apache.org/jira/browse/TIKA-3324
>             Project: Tika
>          Issue Type: Task
>            Reporter: Tim Allison
>            Assignee: Tim Allison
>            Priority: Major
>
> I _think_ we can introduce this gently at first. And slowly fix files as time allows.  Obv, we can hope a bulk fix will work, and it won’t be much effort... WDYT?
>  
> H/T [~ndipiazza_gmail]  for the recommendation.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)