Skip to content

Commit 495f808

Browse files
authored
STYLE: Detect unnecessary pylint ignore (#57918)
1 parent 38086f1 commit 495f808

File tree

5 files changed

+1
-6
lines changed

5 files changed

+1
-6
lines changed

Diff for: .pre-commit-config.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ repos:
7878
hooks:
7979
- id: pylint
8080
stages: [manual]
81-
args: [--load-plugins=pylint.extensions.redefined_loop_name]
81+
args: [--load-plugins=pylint.extensions.redefined_loop_name, --fail-on=I0021]
8282
- id: pylint
8383
alias: redefined-outer-name
8484
name: Redefining name from outer scope

Diff for: pandas/core/groupby/indexing.py

-1
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,6 @@ def _positional_selector(self) -> GroupByPositionalSelector:
114114
4 b 5
115115
"""
116116
if TYPE_CHECKING:
117-
# pylint: disable-next=used-before-assignment
118117
groupby_self = cast(groupby.GroupBy, self)
119118
else:
120119
groupby_self = self

Diff for: pandas/core/tools/datetimes.py

-1
Original file line numberDiff line numberDiff line change
@@ -993,7 +993,6 @@ def to_datetime(
993993
errors=errors,
994994
exact=exact,
995995
)
996-
# pylint: disable-next=used-before-assignment
997996
result: Timestamp | NaTType | Series | Index
998997

999998
if isinstance(arg, Timestamp):

Diff for: pandas/tests/io/formats/test_to_latex.py

-1
Original file line numberDiff line numberDiff line change
@@ -1311,7 +1311,6 @@ def test_to_latex_multiindex_names(self, name0, name1, axes):
13111311
)
13121312
col_names = [n if (bool(n) and 1 in axes) else "" for n in names]
13131313
observed = df.to_latex(multirow=False)
1314-
# pylint: disable-next=consider-using-f-string
13151314
expected = r"""\begin{tabular}{llrrrr}
13161315
\toprule
13171316
& %s & \multicolumn{2}{r}{1} & \multicolumn{2}{r}{2} \\

Diff for: pandas/tests/series/test_iteration.py

-2
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,10 @@ def test_keys(self, datetime_series):
44

55
def test_iter_datetimes(self, datetime_series):
66
for i, val in enumerate(datetime_series):
7-
# pylint: disable-next=unnecessary-list-index-lookup
87
assert val == datetime_series.iloc[i]
98

109
def test_iter_strings(self, string_series):
1110
for i, val in enumerate(string_series):
12-
# pylint: disable-next=unnecessary-list-index-lookup
1311
assert val == string_series.iloc[i]
1412

1513
def test_iteritems_datetimes(self, datetime_series):

0 commit comments

Comments
 (0)