You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "gorewilliams (via GitHub)" <gi...@apache.org> on 2026/08/31 12:32:53 UTC

[PR] Build: Consolidate ruff config into pyproject.toml [iceberg-python]

gorewilliams opened a new pull request, #3883:
URL: https://github.com/apache/iceberg-python/pull/3883

   Closes #3882
   
   Ruff was the only tool still configured outside `pyproject.toml` (`pytest`, `mypy`, `coverage` and `uv` all live there).
   
   The standalone `ruff.toml` was introduced in #619 (April 2024) as a workaround for Dependabot failing to parse the project's `pyproject.toml`. That was the Poetry era; `main` is now PEP 621 metadata with a setuptools backend plus `[tool.uv]`, so the parsing path that motivated the split is no longer in use.
   
   ## Changes
   
   - Move `ruff.toml` contents into `pyproject.toml` under `[tool.ruff]`, `[tool.ruff.lint]`, `[tool.ruff.lint.pyupgrade]`, `[tool.ruff.lint.isort]` and `[tool.ruff.format]`
   - Delete `ruff.toml`
   
   No settings values changed.
   
   ## Verification
   
   - Nothing referenced the file by path — the `ruff`/`ruff-format` and `nbqa-ruff` hooks pass no `--config`, so Ruff's own discovery picks up `[tool.ruff]`.
   - `ruff check --show-settings` output is byte-identical before and after, apart from the `Settings path` line itself.
   - `prek run --all-files` passes.


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [PR] Build: Consolidate ruff config into pyproject.toml [iceberg-python]

Posted by "kevinjqliu (via GitHub)" <gi...@apache.org>.
kevinjqliu merged PR #3883:
URL: https://github.com/apache/iceberg-python/pull/3883


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org