Skip to content

[pylint] - remove AugAssign errors from self-cls-assignment (W0642)#12957

Merged
AlexWaygood merged 3 commits intoastral-sh:mainfrom
diceroll123:remove-augassign-from-plw0642
Aug 18, 2024
Merged

[pylint] - remove AugAssign errors from self-cls-assignment (W0642)#12957
AlexWaygood merged 3 commits intoastral-sh:mainfrom
diceroll123:remove-augassign-from-plw0642

Conversation

@diceroll123
Copy link
Copy Markdown
Contributor

Summary

Fixes #12954

Test Plan

cargo test

Copy link
Copy Markdown
Member

@AlexWaygood AlexWaygood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks good!

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Aug 17, 2024

CodSpeed Performance Report

Merging #12957 will not alter performance

Comparing diceroll123:remove-augassign-from-plw0642 (f982598) with main (900e98b)

Summary

✅ 32 untouched benchmarks

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Aug 17, 2024

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Copy link
Copy Markdown
Member

@AlexWaygood AlexWaygood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@AlexWaygood AlexWaygood added bug Something isn't working rule Implementing or modifying a lint rule labels Aug 18, 2024
@AlexWaygood AlexWaygood enabled auto-merge (squash) August 18, 2024 15:27
@AlexWaygood AlexWaygood merged commit 4881d32 into astral-sh:main Aug 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working rule Implementing or modifying a lint rule

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PLW0642 flags inplace operators called on self

2 participants