Skip to content

Commit

Permalink
ignore _ assignments
Browse files Browse the repository at this point in the history
  • Loading branch information
diceroll123 committed Dec 24, 2023
1 parent 4bbb3c4 commit 6d1d0dd
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 3 deletions.
Expand Up @@ -3,4 +3,4 @@
FIRST, (FIRST, SECOND, (THIRD, FIRST)) = (1, (1, 2)) # PLW0128
FIRST, SECOND, THIRD, FIRST, SECOND = (1, 2, 3, 4) # PLW0128

FIRST, SECOND = (1, 2) # OK
FIRST, SECOND, _, _ = (1, 2, 3, 4) # OK
Expand Up @@ -44,6 +44,10 @@ fn check_expr(checker: &mut Checker, expr: &Expr, names: &mut Vec<String>) {
}
}
Expr::Name(ast::ExprName { id, .. }) => {
if id.to_string() == "_" {
// Ignore `_` assignments
return;
}
if names.contains(&id.to_string()) {
checker.diagnostics.push(Diagnostic::new(
RedeclaredAssignedName {
Expand Down
Expand Up @@ -43,7 +43,7 @@ redeclared_assigned_name.py:4:23: PLW0128 Redeclared variable `FIRST` in assignm
4 | FIRST, SECOND, THIRD, FIRST, SECOND = (1, 2, 3, 4) # PLW0128
| ^^^^^ PLW0128
5 |
6 | FIRST, SECOND = (1, 2) # OK
6 | FIRST, SECOND, _, _ = (1, 2, 3, 4) # OK
|

redeclared_assigned_name.py:4:30: PLW0128 Redeclared variable `SECOND` in assignment
Expand All @@ -53,7 +53,7 @@ redeclared_assigned_name.py:4:30: PLW0128 Redeclared variable `SECOND` in assign
4 | FIRST, SECOND, THIRD, FIRST, SECOND = (1, 2, 3, 4) # PLW0128
| ^^^^^^ PLW0128
5 |
6 | FIRST, SECOND = (1, 2) # OK
6 | FIRST, SECOND, _, _ = (1, 2, 3, 4) # OK
|


0 comments on commit 6d1d0dd

Please sign in to comment.