petern48 commented on PR #2197:
URL: https://github.com/apache/sedona/pull/2197#issuecomment-3137376546
I still think maven spotless apply should run automatically, just that
ideally we don't run it unnecessarily (e.g during python development). Why
don't we just remove this part?
```yaml
always_run:
true # Ensures this hook runs even if no Java files are changed.
# This is useful for spotless:apply which might affect files not
staged.
```
If we remove it, won't pre-commit still run whenever changes to java/scala
files are committed? That's really the only time it matters right IMO. Or am I
missing something? I don't fully understand the intention of the comment.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
