fix(F811): assignment can also be shadowed binding

crates/ruff_linter/resources/test/fixtures/pyflakes/F811_30.py

@@ -0,0 +1,13 @@
+"""Regression test for: https://github.com/astral-sh/ruff/issues/11828"""
+
+
+class A:
+    """A."""
+
+    def foo(self) -> None:
+        """Foo."""
+
+    bar = foo
+
+    def bar(self) -> None:
+        """Bar."""

crates/ruff_linter/src/checkers/ast/analyze/deferred_scopes.rs

@@ -227,6 +227,13 @@ pub(crate) fn deferred_scopes(checker: &mut Checker) {
                             continue;
                         };
 
+                        // Ignore non-import redefinitions
+                        if matches!(shadowed.kind, BindingKind::Assignment)
+                            && matches!(binding.kind, BindingKind::Assignment)
+                        {
+                            continue;
+                        }
+
                         // If this is an overloaded function, abort.
                         if shadowed.kind.is_function_definition() {
                             if checker

crates/ruff_linter/src/rules/pyflakes/mod.rs

@@ -125,6 +125,7 @@ mod tests {
     #[test_case(Rule::RedefinedWhileUnused, Path::new("F811_27.py"))]
     #[test_case(Rule::RedefinedWhileUnused, Path::new("F811_28.py"))]
     #[test_case(Rule::RedefinedWhileUnused, Path::new("F811_29.pyi"))]
+    #[test_case(Rule::RedefinedWhileUnused, Path::new("F811_30.py"))]
     #[test_case(Rule::UndefinedName, Path::new("F821_0.py"))]
     #[test_case(Rule::UndefinedName, Path::new("F821_1.py"))]
     #[test_case(Rule::UndefinedName, Path::new("F821_2.py"))]

crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__F811_F811_30.py.snap

@@ -0,0 +1,12 @@
+---
+source: crates/ruff_linter/src/rules/pyflakes/mod.rs
+---
+F811_30.py:12:9: F811 Redefinition of unused `bar` from line 10
+   |
+10 |     bar = foo
+11 | 
+12 |     def bar(self) -> None:
+   |         ^^^ F811
+13 |         """Bar."""
+   |
+   = help: Remove definition: `bar`

crates/ruff_python_semantic/src/binding.rs

@@ -179,14 +179,15 @@ impl<'a> Binding<'a> {
             }
             _ => {}
         }
-        // Otherwise, the shadowed binding must be a class definition, function definition, or
-        // import to be considered a redefinition.
+        // Otherwise, the shadowed binding must be a class definition, function definition,
+        // import, or assignment to be considered a redefinition.
         matches!(
             existing.kind,
             BindingKind::ClassDefinition(_)
                 | BindingKind::FunctionDefinition(_)
                 | BindingKind::Import(_)
                 | BindingKind::FromImport(_)
+                | BindingKind::Assignment
         )
     }