Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove deprecated CLI option --format #10170

Merged
merged 1 commit into from Feb 29, 2024
Merged

Conversation

MichaReiser
Copy link
Member

@MichaReiser MichaReiser commented Feb 29, 2024

Summary

Extracted from #9814

Removes the deprecated CLI option 'format' in favor of output-format. The option was deprecated in #8203

Test Plan

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
error: unexpected argument '--format' found

  tip: to pass '--format' as a value, use '-- --format'

@MichaReiser MichaReiser added this to the v0.3.0 milestone Feb 29, 2024
@MichaReiser MichaReiser added breaking Breaking API change cli Related to the command-line interface labels Feb 29, 2024
@MichaReiser MichaReiser marked this pull request as ready for review February 29, 2024 13:54
@MichaReiser
Copy link
Member Author

Thanks @tibor-reiss

@MichaReiser MichaReiser enabled auto-merge (squash) February 29, 2024 13:57
@MichaReiser MichaReiser merged commit 52f4c1e into main Feb 29, 2024
17 checks passed
@MichaReiser MichaReiser deleted the remove-format-cli-option branch February 29, 2024 13:59
Copy link

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

nkxxll pushed a commit to nkxxll/ruff that referenced this pull request Mar 10, 2024
Co-authored-by: Tibor Reiss <tibor.reiss@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Breaking API change cli Related to the command-line interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants