jiayuasu opened a new pull request, #2848: URL: https://github.com/apache/sedona/pull/2848
## Did you read the Contributor Guide? - Yes, I have read the [Contributor Guide](https://sedona.apache.org/latest/community/rule/). ## Is this PR related to a JIRA ticket? No — trivial CI config cleanup. ## What changes were proposed in this PR? The `forbid-tabs` and `remove-tabs` pre-commit hooks currently list specific Makefile paths in their `exclude` regex: ``` ^Makefile$|^python/sedona/doc/Makefile$ ``` This only exempts the repo-root Makefile and the one under `python/sedona/doc/`. Any new Makefile added elsewhere in the tree (e.g. in a sub-tester module) will: 1. Trip `forbid-tabs`, which fails CI because Makefiles must use tabs. 2. Get rewritten by `remove-tabs` (tabs → 2 spaces), which then trips `editorconfig-checker` because the `.editorconfig` for Makefiles requires tabs. This PR replaces the two path-specific entries with a single `(^|/)Makefile$` pattern that excludes any Makefile anywhere in the tree. Behavior on the two existing Makefiles is unchanged; it just future-proofs the config. ## How was this patch tested? Ran `pre-commit run forbid-tabs remove-tabs editorconfig-checker --all-files` locally — all pass. ## Did this PR include necessary documentation updates? - No, this PR does not affect any public API or user-facing behavior. -- 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]
