Skip to content

Comments

[ruff] Exempt NewType calls where the original type is immutable (RUF009)#15588

Merged
dhruvmanila merged 3 commits intoastral-sh:mainfrom
InSyncWithFoo:RUF009
Jan 20, 2025
Merged

[ruff] Exempt NewType calls where the original type is immutable (RUF009)#15588
dhruvmanila merged 3 commits intoastral-sh:mainfrom
InSyncWithFoo:RUF009

Conversation

@InSyncWithFoo
Copy link
Contributor

Summary

Resolves #6447.

Test Plan

cargo nextest run and cargo insta test.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 20, 2025

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@dhruvmanila dhruvmanila added the bug Something isn't working label Jan 20, 2025
@dhruvmanila dhruvmanila merged commit 4cfa355 into astral-sh:main Jan 20, 2025
21 checks passed
@InSyncWithFoo InSyncWithFoo deleted the RUF009 branch January 20, 2025 14:51
dhruvmanila added a commit that referenced this pull request Jan 29, 2025
…immutable (`B008`) (#15765)

## Summary

Resolves #12717.

This change incorporates the logic added in #15588.

## Test Plan

`cargo nextest run` and `cargo insta test`.

---------

Co-authored-by: Dhruv Manilawala <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

False positive with RUF009 using NewType, Final and dataclass

3 participants