mirror of
https://github.com/astral-sh/ruff.git
synced 2025-09-27 04:19:18 +00:00
[ruff] Stabilize unsafe fix for zip-instead-of-pairwise (RUF007) (#14401)
This PR stabilizes the unsafe fix for [zip-instead-of-pairwise (RUF007)](https://docs.astral.sh/ruff/rules/zip-instead-of-pairwise/#zip-instead-of-pairwise-ruf007), which replaces the use of zip with that of itertools.pairwise and has been available under preview since version 0.5.7. There are no open issues regarding RUF007 at the time of this writing.
This commit is contained in:
Dylan
Micha Reiser
parent
5f6607bf54
commit
c0b3dd3745
4 changed files with 164 additions and 277 deletions
|
|
@ -390,7 +390,6 @@ mod tests {
|
|||
Ok(())
|
||||
}
|
||||
|
||||
#[test_case(Rule::ZipInsteadOfPairwise, Path::new("RUF007.py"))]
|
||||
#[test_case(Rule::UnsafeMarkupUse, Path::new("RUF035.py"))]
|
||||
#[test_case(
|
||||
Rule::FunctionCallInDataclassDefaultArgument,
|
||||
|
|
|
|||
|
|
@ -154,18 +154,16 @@ pub(crate) fn zip_instead_of_pairwise(checker: &mut Checker, call: &ast::ExprCal
|
|||
|
||||
let mut diagnostic = Diagnostic::new(ZipInsteadOfPairwise, func.range());
|
||||
|
||||
if checker.settings.preview.is_enabled() {
|
||||
diagnostic.try_set_fix(|| {
|
||||
let (import_edit, binding) = checker.importer().get_or_import_symbol(
|
||||
&ImportRequest::import("itertools", "pairwise"),
|
||||
func.start(),
|
||||
checker.semantic(),
|
||||
)?;
|
||||
let reference_edit =
|
||||
Edit::range_replacement(format!("{binding}({})", first_arg_info.id), call.range());
|
||||
Ok(Fix::unsafe_edits(import_edit, [reference_edit]))
|
||||
});
|
||||
}
|
||||
diagnostic.try_set_fix(|| {
|
||||
let (import_edit, binding) = checker.importer().get_or_import_symbol(
|
||||
&ImportRequest::import("itertools", "pairwise"),
|
||||
func.start(),
|
||||
checker.semantic(),
|
||||
)?;
|
||||
let reference_edit =
|
||||
Edit::range_replacement(format!("{binding}({})", first_arg_info.id), call.range());
|
||||
Ok(Fix::unsafe_edits(import_edit, [reference_edit]))
|
||||
});
|
||||
|
||||
checker.diagnostics.push(diagnostic);
|
||||
}
|
||||
|
|
|
|||
|
|
@ -1,8 +1,7 @@
|
|||
---
|
||||
source: crates/ruff_linter/src/rules/ruff/mod.rs
|
||||
snapshot_kind: text
|
||||
---
|
||||
RUF007.py:16:1: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
RUF007.py:16:1: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
15 | # Errors
|
||||
16 | zip(input, input[1:])
|
||||
|
|
@ -12,7 +11,22 @@ RUF007.py:16:1: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating
|
|||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
RUF007.py:17:1: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
13 14 | zip(foo[:-1], foo[1:], foo, strict=True) # more than 2 inputs
|
||||
14 15 |
|
||||
15 16 | # Errors
|
||||
16 |-zip(input, input[1:])
|
||||
17 |+itertools.pairwise(input)
|
||||
17 18 | zip(input, input[1::1])
|
||||
18 19 | zip(input[:-1], input[1:])
|
||||
19 20 | zip(input[1:], input[2:])
|
||||
|
||||
RUF007.py:17:1: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
15 | # Errors
|
||||
16 | zip(input, input[1:])
|
||||
|
|
@ -23,7 +37,22 @@ RUF007.py:17:1: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating
|
|||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
RUF007.py:18:1: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
14 15 |
|
||||
15 16 | # Errors
|
||||
16 17 | zip(input, input[1:])
|
||||
17 |-zip(input, input[1::1])
|
||||
18 |+itertools.pairwise(input)
|
||||
18 19 | zip(input[:-1], input[1:])
|
||||
19 20 | zip(input[1:], input[2:])
|
||||
20 21 | zip(input[1:-1], input[2:])
|
||||
|
||||
RUF007.py:18:1: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
16 | zip(input, input[1:])
|
||||
17 | zip(input, input[1::1])
|
||||
|
|
@ -34,7 +63,22 @@ RUF007.py:18:1: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating
|
|||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
RUF007.py:19:1: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
15 16 | # Errors
|
||||
16 17 | zip(input, input[1:])
|
||||
17 18 | zip(input, input[1::1])
|
||||
18 |-zip(input[:-1], input[1:])
|
||||
19 |+itertools.pairwise(input)
|
||||
19 20 | zip(input[1:], input[2:])
|
||||
20 21 | zip(input[1:-1], input[2:])
|
||||
21 22 | list(zip(input, input[1:]))
|
||||
|
||||
RUF007.py:19:1: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
17 | zip(input, input[1::1])
|
||||
18 | zip(input[:-1], input[1:])
|
||||
|
|
@ -45,7 +89,22 @@ RUF007.py:19:1: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating
|
|||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
RUF007.py:20:1: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
16 17 | zip(input, input[1:])
|
||||
17 18 | zip(input, input[1::1])
|
||||
18 19 | zip(input[:-1], input[1:])
|
||||
19 |-zip(input[1:], input[2:])
|
||||
20 |+itertools.pairwise(input)
|
||||
20 21 | zip(input[1:-1], input[2:])
|
||||
21 22 | list(zip(input, input[1:]))
|
||||
22 23 | list(zip(input[:-1], input[1:]))
|
||||
|
||||
RUF007.py:20:1: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
18 | zip(input[:-1], input[1:])
|
||||
19 | zip(input[1:], input[2:])
|
||||
|
|
@ -56,7 +115,22 @@ RUF007.py:20:1: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating
|
|||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
RUF007.py:21:6: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
17 18 | zip(input, input[1::1])
|
||||
18 19 | zip(input[:-1], input[1:])
|
||||
19 20 | zip(input[1:], input[2:])
|
||||
20 |-zip(input[1:-1], input[2:])
|
||||
21 |+itertools.pairwise(input)
|
||||
21 22 | list(zip(input, input[1:]))
|
||||
22 23 | list(zip(input[:-1], input[1:]))
|
||||
23 24 | zip(foo[:-1], foo[1:], strict=True)
|
||||
|
||||
RUF007.py:21:6: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
19 | zip(input[1:], input[2:])
|
||||
20 | zip(input[1:-1], input[2:])
|
||||
|
|
@ -67,7 +141,22 @@ RUF007.py:21:6: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating
|
|||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
RUF007.py:22:6: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
18 19 | zip(input[:-1], input[1:])
|
||||
19 20 | zip(input[1:], input[2:])
|
||||
20 21 | zip(input[1:-1], input[2:])
|
||||
21 |-list(zip(input, input[1:]))
|
||||
22 |+list(itertools.pairwise(input))
|
||||
22 23 | list(zip(input[:-1], input[1:]))
|
||||
23 24 | zip(foo[:-1], foo[1:], strict=True)
|
||||
24 25 | zip(foo[:-1], foo[1:], strict=False)
|
||||
|
||||
RUF007.py:22:6: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
20 | zip(input[1:-1], input[2:])
|
||||
21 | list(zip(input, input[1:]))
|
||||
|
|
@ -78,7 +167,22 @@ RUF007.py:22:6: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating
|
|||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
RUF007.py:23:1: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
19 20 | zip(input[1:], input[2:])
|
||||
20 21 | zip(input[1:-1], input[2:])
|
||||
21 22 | list(zip(input, input[1:]))
|
||||
22 |-list(zip(input[:-1], input[1:]))
|
||||
23 |+list(itertools.pairwise(input))
|
||||
23 24 | zip(foo[:-1], foo[1:], strict=True)
|
||||
24 25 | zip(foo[:-1], foo[1:], strict=False)
|
||||
25 26 | zip(foo[:-1], foo[1:], strict=bool(foo))
|
||||
|
||||
RUF007.py:23:1: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
21 | list(zip(input, input[1:]))
|
||||
22 | list(zip(input[:-1], input[1:]))
|
||||
|
|
@ -89,7 +193,21 @@ RUF007.py:23:1: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating
|
|||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
RUF007.py:24:1: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
20 21 | zip(input[1:-1], input[2:])
|
||||
21 22 | list(zip(input, input[1:]))
|
||||
22 23 | list(zip(input[:-1], input[1:]))
|
||||
23 |-zip(foo[:-1], foo[1:], strict=True)
|
||||
24 |+itertools.pairwise(foo)
|
||||
24 25 | zip(foo[:-1], foo[1:], strict=False)
|
||||
25 26 | zip(foo[:-1], foo[1:], strict=bool(foo))
|
||||
|
||||
RUF007.py:24:1: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
22 | list(zip(input[:-1], input[1:]))
|
||||
23 | zip(foo[:-1], foo[1:], strict=True)
|
||||
|
|
@ -99,7 +217,20 @@ RUF007.py:24:1: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating
|
|||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
RUF007.py:25:1: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
21 22 | list(zip(input, input[1:]))
|
||||
22 23 | list(zip(input[:-1], input[1:]))
|
||||
23 24 | zip(foo[:-1], foo[1:], strict=True)
|
||||
24 |-zip(foo[:-1], foo[1:], strict=False)
|
||||
25 |+itertools.pairwise(foo)
|
||||
25 26 | zip(foo[:-1], foo[1:], strict=bool(foo))
|
||||
|
||||
RUF007.py:25:1: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
23 | zip(foo[:-1], foo[1:], strict=True)
|
||||
24 | zip(foo[:-1], foo[1:], strict=False)
|
||||
|
|
@ -107,3 +238,15 @@ RUF007.py:25:1: RUF007 Prefer `itertools.pairwise()` over `zip()` when iterating
|
|||
| ^^^ RUF007
|
||||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
22 23 | list(zip(input[:-1], input[1:]))
|
||||
23 24 | zip(foo[:-1], foo[1:], strict=True)
|
||||
24 25 | zip(foo[:-1], foo[1:], strict=False)
|
||||
25 |-zip(foo[:-1], foo[1:], strict=bool(foo))
|
||||
26 |+itertools.pairwise(foo)
|
||||
|
|
|
|||
|
|
@ -1,253 +0,0 @@
|
|||
---
|
||||
source: crates/ruff_linter/src/rules/ruff/mod.rs
|
||||
snapshot_kind: text
|
||||
---
|
||||
RUF007.py:16:1: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
15 | # Errors
|
||||
16 | zip(input, input[1:])
|
||||
| ^^^ RUF007
|
||||
17 | zip(input, input[1::1])
|
||||
18 | zip(input[:-1], input[1:])
|
||||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
13 14 | zip(foo[:-1], foo[1:], foo, strict=True) # more than 2 inputs
|
||||
14 15 |
|
||||
15 16 | # Errors
|
||||
16 |-zip(input, input[1:])
|
||||
17 |+itertools.pairwise(input)
|
||||
17 18 | zip(input, input[1::1])
|
||||
18 19 | zip(input[:-1], input[1:])
|
||||
19 20 | zip(input[1:], input[2:])
|
||||
|
||||
RUF007.py:17:1: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
15 | # Errors
|
||||
16 | zip(input, input[1:])
|
||||
17 | zip(input, input[1::1])
|
||||
| ^^^ RUF007
|
||||
18 | zip(input[:-1], input[1:])
|
||||
19 | zip(input[1:], input[2:])
|
||||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
14 15 |
|
||||
15 16 | # Errors
|
||||
16 17 | zip(input, input[1:])
|
||||
17 |-zip(input, input[1::1])
|
||||
18 |+itertools.pairwise(input)
|
||||
18 19 | zip(input[:-1], input[1:])
|
||||
19 20 | zip(input[1:], input[2:])
|
||||
20 21 | zip(input[1:-1], input[2:])
|
||||
|
||||
RUF007.py:18:1: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
16 | zip(input, input[1:])
|
||||
17 | zip(input, input[1::1])
|
||||
18 | zip(input[:-1], input[1:])
|
||||
| ^^^ RUF007
|
||||
19 | zip(input[1:], input[2:])
|
||||
20 | zip(input[1:-1], input[2:])
|
||||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
15 16 | # Errors
|
||||
16 17 | zip(input, input[1:])
|
||||
17 18 | zip(input, input[1::1])
|
||||
18 |-zip(input[:-1], input[1:])
|
||||
19 |+itertools.pairwise(input)
|
||||
19 20 | zip(input[1:], input[2:])
|
||||
20 21 | zip(input[1:-1], input[2:])
|
||||
21 22 | list(zip(input, input[1:]))
|
||||
|
||||
RUF007.py:19:1: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
17 | zip(input, input[1::1])
|
||||
18 | zip(input[:-1], input[1:])
|
||||
19 | zip(input[1:], input[2:])
|
||||
| ^^^ RUF007
|
||||
20 | zip(input[1:-1], input[2:])
|
||||
21 | list(zip(input, input[1:]))
|
||||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
16 17 | zip(input, input[1:])
|
||||
17 18 | zip(input, input[1::1])
|
||||
18 19 | zip(input[:-1], input[1:])
|
||||
19 |-zip(input[1:], input[2:])
|
||||
20 |+itertools.pairwise(input)
|
||||
20 21 | zip(input[1:-1], input[2:])
|
||||
21 22 | list(zip(input, input[1:]))
|
||||
22 23 | list(zip(input[:-1], input[1:]))
|
||||
|
||||
RUF007.py:20:1: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
18 | zip(input[:-1], input[1:])
|
||||
19 | zip(input[1:], input[2:])
|
||||
20 | zip(input[1:-1], input[2:])
|
||||
| ^^^ RUF007
|
||||
21 | list(zip(input, input[1:]))
|
||||
22 | list(zip(input[:-1], input[1:]))
|
||||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
17 18 | zip(input, input[1::1])
|
||||
18 19 | zip(input[:-1], input[1:])
|
||||
19 20 | zip(input[1:], input[2:])
|
||||
20 |-zip(input[1:-1], input[2:])
|
||||
21 |+itertools.pairwise(input)
|
||||
21 22 | list(zip(input, input[1:]))
|
||||
22 23 | list(zip(input[:-1], input[1:]))
|
||||
23 24 | zip(foo[:-1], foo[1:], strict=True)
|
||||
|
||||
RUF007.py:21:6: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
19 | zip(input[1:], input[2:])
|
||||
20 | zip(input[1:-1], input[2:])
|
||||
21 | list(zip(input, input[1:]))
|
||||
| ^^^ RUF007
|
||||
22 | list(zip(input[:-1], input[1:]))
|
||||
23 | zip(foo[:-1], foo[1:], strict=True)
|
||||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
18 19 | zip(input[:-1], input[1:])
|
||||
19 20 | zip(input[1:], input[2:])
|
||||
20 21 | zip(input[1:-1], input[2:])
|
||||
21 |-list(zip(input, input[1:]))
|
||||
22 |+list(itertools.pairwise(input))
|
||||
22 23 | list(zip(input[:-1], input[1:]))
|
||||
23 24 | zip(foo[:-1], foo[1:], strict=True)
|
||||
24 25 | zip(foo[:-1], foo[1:], strict=False)
|
||||
|
||||
RUF007.py:22:6: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
20 | zip(input[1:-1], input[2:])
|
||||
21 | list(zip(input, input[1:]))
|
||||
22 | list(zip(input[:-1], input[1:]))
|
||||
| ^^^ RUF007
|
||||
23 | zip(foo[:-1], foo[1:], strict=True)
|
||||
24 | zip(foo[:-1], foo[1:], strict=False)
|
||||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
19 20 | zip(input[1:], input[2:])
|
||||
20 21 | zip(input[1:-1], input[2:])
|
||||
21 22 | list(zip(input, input[1:]))
|
||||
22 |-list(zip(input[:-1], input[1:]))
|
||||
23 |+list(itertools.pairwise(input))
|
||||
23 24 | zip(foo[:-1], foo[1:], strict=True)
|
||||
24 25 | zip(foo[:-1], foo[1:], strict=False)
|
||||
25 26 | zip(foo[:-1], foo[1:], strict=bool(foo))
|
||||
|
||||
RUF007.py:23:1: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
21 | list(zip(input, input[1:]))
|
||||
22 | list(zip(input[:-1], input[1:]))
|
||||
23 | zip(foo[:-1], foo[1:], strict=True)
|
||||
| ^^^ RUF007
|
||||
24 | zip(foo[:-1], foo[1:], strict=False)
|
||||
25 | zip(foo[:-1], foo[1:], strict=bool(foo))
|
||||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
20 21 | zip(input[1:-1], input[2:])
|
||||
21 22 | list(zip(input, input[1:]))
|
||||
22 23 | list(zip(input[:-1], input[1:]))
|
||||
23 |-zip(foo[:-1], foo[1:], strict=True)
|
||||
24 |+itertools.pairwise(foo)
|
||||
24 25 | zip(foo[:-1], foo[1:], strict=False)
|
||||
25 26 | zip(foo[:-1], foo[1:], strict=bool(foo))
|
||||
|
||||
RUF007.py:24:1: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
22 | list(zip(input[:-1], input[1:]))
|
||||
23 | zip(foo[:-1], foo[1:], strict=True)
|
||||
24 | zip(foo[:-1], foo[1:], strict=False)
|
||||
| ^^^ RUF007
|
||||
25 | zip(foo[:-1], foo[1:], strict=bool(foo))
|
||||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
21 22 | list(zip(input, input[1:]))
|
||||
22 23 | list(zip(input[:-1], input[1:]))
|
||||
23 24 | zip(foo[:-1], foo[1:], strict=True)
|
||||
24 |-zip(foo[:-1], foo[1:], strict=False)
|
||||
25 |+itertools.pairwise(foo)
|
||||
25 26 | zip(foo[:-1], foo[1:], strict=bool(foo))
|
||||
|
||||
RUF007.py:25:1: RUF007 [*] Prefer `itertools.pairwise()` over `zip()` when iterating over successive pairs
|
||||
|
|
||||
23 | zip(foo[:-1], foo[1:], strict=True)
|
||||
24 | zip(foo[:-1], foo[1:], strict=False)
|
||||
25 | zip(foo[:-1], foo[1:], strict=bool(foo))
|
||||
| ^^^ RUF007
|
||||
|
|
||||
= help: Replace `zip()` with `itertools.pairwise()`
|
||||
|
||||
ℹ Unsafe fix
|
||||
1 |+import itertools
|
||||
1 2 | input = [1, 2, 3]
|
||||
2 3 | otherInput = [2, 3, 4]
|
||||
3 4 | foo = [1, 2, 3, 4]
|
||||
--------------------------------------------------------------------------------
|
||||
22 23 | list(zip(input[:-1], input[1:]))
|
||||
23 24 | zip(foo[:-1], foo[1:], strict=True)
|
||||
24 25 | zip(foo[:-1], foo[1:], strict=False)
|
||||
25 |-zip(foo[:-1], foo[1:], strict=bool(foo))
|
||||
26 |+itertools.pairwise(foo)
|
||||
Loading…
Add table
Add a link
Reference in a new issue