Update w.r.t latest API changes

This commit is contained in:
Dhruv Manilawala 2023-07-13 20:55:15 +05:30
parent 4110065a18
commit a3b4adab3f
No known key found for this signature in database
GPG Key ID: 9601911DE8797432
2 changed files with 56 additions and 2 deletions

View File

@ -116,8 +116,8 @@ pub(crate) fn unused_import(checker: &Checker, scope: &Scope, diagnostics: &mut
};
for binding in scope
.bindings_for_name(binding_name)
.map(|binding_id| &checker.semantic_model().bindings[binding_id])
.get_all(binding_name)
.map(|binding_id| checker.semantic().binding(binding_id))
{
let Some(qualified_name) = binding.qualified_name() else {
break;

View File

@ -0,0 +1,54 @@
---
source: crates/ruff/src/rules/pyflakes/mod.rs
---
F401_20.py:7:8: F401 [*] `multiprocessing.connection` imported but unused
|
5 | multiprocessing = None
6 |
7 | import multiprocessing.connection
| ^^^^^^^^^^^^^^^^^^^^^^^^^^ F401
8 | import multiprocessing.pool
9 | import multiprocessing.queues
|
= help: Remove unused import: `multiprocessing.connection`
Fix
4 4 | # imports. It should only detect imports, not any other kind of binding.
5 5 | multiprocessing = None
6 6 |
7 |-import multiprocessing.connection
8 7 | import multiprocessing.pool
9 8 | import multiprocessing.queues
F401_20.py:8:8: F401 [*] `multiprocessing.pool` imported but unused
|
7 | import multiprocessing.connection
8 | import multiprocessing.pool
| ^^^^^^^^^^^^^^^^^^^^ F401
9 | import multiprocessing.queues
|
= help: Remove unused import: `multiprocessing.pool`
Fix
5 5 | multiprocessing = None
6 6 |
7 7 | import multiprocessing.connection
8 |-import multiprocessing.pool
9 8 | import multiprocessing.queues
F401_20.py:9:8: F401 [*] `multiprocessing.queues` imported but unused
|
7 | import multiprocessing.connection
8 | import multiprocessing.pool
9 | import multiprocessing.queues
| ^^^^^^^^^^^^^^^^^^^^^^ F401
|
= help: Remove unused import: `multiprocessing.queues`
Fix
6 6 |
7 7 | import multiprocessing.connection
8 8 | import multiprocessing.pool
9 |-import multiprocessing.queues