rmuir opened a new pull request, #14825:
URL: https://github.com/apache/lucene/pull/14825

   You generally shouldn't need the check, as your editor should follow the 
editorconfig and not introduce the errors checked-for here.
   
   But it helps maintain the .editorconfig itself and ensure that files in the 
repository actually match whatever is configured there.
   
   If you want to enable the check locally for debugging:
   ```
   GOBIN=${HOME}/.local/bin go install gitlab.com/greut/eclint/cmd/eclint@latest
   echo "lucene.tool.eclint=eclint" >> ~/.gradle/gradle.properties
   ```
   
   The eclint action is not used, as it is not on the apache approved list. 
Getting the 3MB binary in CI takes 2s (cached) and 9s (uncached, using 
setup-go).
   
   The check itself runs fast:
   ```
   real 0m3.647s
   user 0m6.702s
   sys  0m0.474s
   ```
   
   @dweiss, I imagine you want to keep things a certain way, please feel free 
to push commits here. I just hack when it comes to the gradle side of things, 
but the actions part is decent I think.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to