Skip to content

Use rule name rather than message in --statistics - #11697

Merged
MichaReiser merged 2 commits into
ruff-0.5from
charlie/name
Jun 24, 2024
Merged

MichaReiser merged 2 commits into
ruff-0.5from
charlie/name

Conversation

@charliermarsh

Copy link
Copy Markdown
Member

Summary

Since this changes the output schema of the JSON format for --statistics, let's include it in v0.5.0.

Closes #11097.

@charliermarsh charliermarsh added breaking Backwards-incompatible changes cli Related to the command-line interface of Ruff or ty labels Jun 2, 2024
@charliermarsh charliermarsh added this to the v0.5.0 milestone Jun 2, 2024
@zanieb
zanieb self-requested a review June 2, 2024 18:29
@github-actions

github-actions Bot commented Jun 2, 2024 •

Copy link
Copy Markdown
Contributor

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Comment thread crates/ruff/tests/integration_test.rs
@MichaReiser
MichaReiser changed the base branch from main to ruff-0.5 June 24, 2024 12:00
@MichaReiser
MichaReiser merged commit 7096340 into ruff-0.5 Jun 24, 2024
@MichaReiser
MichaReiser deleted the charlie/name branch June 24, 2024 12:23
@MichaReiser MichaReiser mentioned this pull request Jun 26, 2024
MichaReiser added a commit that referenced this pull request Jun 27, 2024
Co-authored-by: Micha Reiser <micha@reiser.io>
Closes #11097.
MichaReiser added a commit that referenced this pull request Jun 27, 2024
Co-authored-by: Micha Reiser <micha@reiser.io>
Closes #11097.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Backwards-incompatible changes cli Related to the command-line interface of Ruff or ty

Projects

None yet

Development

Successfully merging this pull request may close these issues.

idea: add generic name to rule violations if message contains formatted strings when output in --statistics

3 participants