ruff
ruff copied to clipboard
Remove deprecated configuration '--show-source`
Fixes parts of https://github.com/astral-sh/ruff/issues/7650:
Extracted from this PR
1. removes the deprecated configuration 'format'. It was deprecated in https://github.com/astral-sh/ruff/pull/8203 in favor of 'output-format'. Update: Was merged in https://github.com/astral-sh/ruff/pull/10170 Tests: - `cargo run -p ruff -- --explain RET505` works - `cargo run -p ruff -- --explain RET505 --output-format json` works - `cargo run -p ruff -- --explain RET505 --format json` does not work anymore- removes deprecated configurations 'show-source' and 'no-show-source'. Tests:
cargo run -p ruff -- --show-sourcedoes not work anymorecargo run -p ruff -- --no-show-sourcedoes not work anymore
resolves: #7349
ruff-ecosystem results
Linter (stable)
✅ ecosystem check detected no linter changes.
Linter (preview)
✅ ecosystem check detected no linter changes.
Thanks for your contribution! We won't be able to release this until v0.3.0 so it's going to be a bit before we can merge
Hi @zanieb, since there are more of these deprecations in the codebase, do you mind if I keep adding commits to this PR, or shall I open a separate one for each of them?
Hey @tibor-reiss — I'm unsure. Ideally these things would be in separate pull requests for our changelog and review but we're talking pretty far out here and it could be a significant amount to keep your changes up to date with main. In general, I'd recommend pursuing issues that aren't breaking changes since they aren't timing sensitive.
Another thing we can do is have deprecated options error in preview mode first but retain a warning in stable.
Hi @zanieb, I understand. I added the 2nd commit just before your answer - talk about timing. And as you predicted, merge conflict. Since the timeline is unclear, and this is anyways a low-prio change, I would recommend gathering the commits here and not polluting the board with more PRs. Once we get closer, it's a minimal change to pull out the commits into separate PRs so you can ping me about your preferences about this change. What do you think?
@tibor-reiss I extracted the first breaking change into its own PR and ship it as part of 0.3. We want to wait a little longer with the other options because it isn't that long ago that we've deprecated them.