Skip to content

Respect per-file-ignores for RUF100 with no other diagnostics#11058

Merged
charliermarsh merged 1 commit intomainfrom
charlie/per
Apr 20, 2024
Merged

Respect per-file-ignores for RUF100 with no other diagnostics#11058
charliermarsh merged 1 commit intomainfrom
charlie/per

Conversation

@charliermarsh
Copy link
Member

Summary

The existing test didn't cover the case in which there are no other diagnostics in the file.

Closes #10906.

@charliermarsh charliermarsh added bug Something isn't working suppression Related to supression of violations e.g. noqa labels Apr 20, 2024
@charliermarsh charliermarsh enabled auto-merge (squash) April 20, 2024 15:26
@charliermarsh charliermarsh merged commit d544199 into main Apr 20, 2024
@charliermarsh charliermarsh deleted the charlie/per branch April 20, 2024 15:33
@github-actions
Copy link
Contributor

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working suppression Related to supression of violations e.g. noqa

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RUF100 error despite listed in per-file-ignores

1 participant