NoahKusaba opened a new pull request, #2472: URL: https://github.com/apache/datafusion-ballista/pull/2472
# Rationale for this change The ruff step in `build.yml` ran only inside `python/`, so twelve Python files were never linted, including every `dev/` script, the benchmark scripts and `docs/source/conf.py`. # What changes are included in this PR? - Run `ruff check` and `ruff format --check` from the repository root. Ruff uses the nearest config per file, so `python/` keeps its `pyproject.toml`. - Add a root `ruff.toml` for files outside `python/`. `E402` is ignored per-file in the benchmark scripts, which deliberately run code before their imports. - Reformat the newly covered files and fix the real lint findings this surfaced. The reformat-only files were checked by comparing parsed ASTs before and after. # Are there any user-facing changes? No. Contributors editing Python outside `python/` will now get ruff findings in CI. Verified with ruff 0.14.13 (the version CI installs), run both from the root and from `python/` against `..` as CI does: 22 files, clean. Split out of #2438 to make it easier to review. 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
