You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tika.apache.org by "Tim Allison (Jira)" <ji...@apache.org> on 2025/12/03 21:32:00 UTC

[jira] [Resolved] (TIKA-4251) [DISCUSS] move to cosium's git-code-format-maven-plugin with google-java-format

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

Tim Allison resolved TIKA-4251.
-------------------------------
    Fix Version/s: 4.0.0
                   3.3.0
       Resolution: Fixed

> [DISCUSS] move to cosium's git-code-format-maven-plugin with google-java-format
> -------------------------------------------------------------------------------
>
>                 Key: TIKA-4251
>                 URL: https://issues.apache.org/jira/browse/TIKA-4251
>             Project: Tika
>          Issue Type: Task
>            Reporter: Tim Allison
>            Priority: Major
>             Fix For: 4.0.0, 3.3.0
>
>         Attachments: spotless3-2025.12.03.diff, tika-app.diff
>
>
> I was recently working a bit on incubator-stormcrawler, and I noticed that they are using cosium's git-code-format-maven-plugin: https://github.com/Cosium/git-code-format-maven-plugin
> I was initially annoyed that I couldn't quickly figure out what I had to fix to make the linter happyl, but then I realized there was a magic command: {{mvn git-code-format:format-code}} which just fixed the code so that the linter passed. 
> The one drawback I found is that it does not fix nor does it alert on wildcard imports.  We could still use checkstyle for that but only have one rule for checkstyle.
> The other drawback is that there is not a lot of room for variation from google's style. This may actually be a benefit, too, of course.
> I just ran this on {{tika-core}} here: https://github.com/apache/tika/tree/google-java-format
> What would you think about making this change for 3.x?



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