andygrove opened a new pull request, #3943: URL: https://github.com/apache/datafusion-comet/pull/3943
## Which issue does this PR close? N/A - documentation improvement ## Rationale for this change New Scala test suites must be manually added to the CI workflow files (`pr_build_linux.yml` and `pr_build_macos.yml`) so they run in CI. This is not documented anywhere, so contributors may add test suites that pass locally but never run in CI. ## What changes are included in this PR? Adds a new section to the contributor guide (`docs/source/contributor-guide/development.md`) explaining: - That CI does not auto-discover test suites — they must be listed explicitly in both workflow files - Why suites are grouped by category (parallel execution to reduce CI time) - How to add a new suite, with a YAML example - A table mapping each group name (`fuzz`, `shuffle`, `parquet`, `csv`, `exec`, `expressions`, `sql`) to the area it covers - That both workflow files must stay in sync ## How are these changes tested? Documentation-only change. Verified the markdown renders correctly. -- 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]
