andygrove opened a new pull request, #3664: URL: https://github.com/apache/datafusion-comet/pull/3664
## Summary - Add a new lightweight CI workflow (`pr_rat_check.yml`) that runs the Apache RAT license header check on **all** PRs, including docs-only changes ## Problem The existing build workflows (`pr_build_linux`, `spark_sql_test`, etc.) use `paths-ignore` to skip docs-only changes (matching `docs/**`, `**.md`, etc.). This also skips the RAT license check, which allowed files without Apache license headers to be merged via docs-only PRs (e.g., #3651 introduced SVG files without headers, breaking CI for unrelated PRs like #3661). ## Solution A dedicated workflow that: - Has **no path filters** — runs on every PR and push to main - Only runs `./mvnw -B -N apache-rat:check` (the `-N` flag skips child module builds) - Uses a minimal setup (just Java 11, no Rust/Spark needed) - Completes in seconds since it only checks license headers ## Test plan - [x] The workflow itself will be validated by CI on this PR -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
