Skip to content

Commit

Permalink
Follow up to fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
DanielNoord authored Jan 12, 2025
1 parent 6d558c7 commit 8d8f988
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions pylint/lint/pylinter.py
Original file line number Diff line number Diff line change
Expand Up @@ -261,11 +261,10 @@ def _load_reporter_by_class(reporter_class: str) -> type[BaseReporter]:


def _handle_force_color_no_color(
reporters: list[reporters.BaseReporter],
reporters: Sequence[BaseReporter],

Check warning on line 264 in pylint/lint/pylinter.py

View workflow job for this annotation

GitHub Actions / pylint

W0621

Redefining name 'reporters' from outer scope (line 27)
) -> list[reporters.BaseReporter]:
"""
Check ``NO_COLOR`` and ``FORCE_COLOR``, and return the modified reporter list
accordingly.
Check ``NO_COLOR`` and ``FORCE_COLOR``, and return the modified reporter list.
Rules are presented in this table:
+--------------+---------------+-----------------+------------------------------------------------------------+
Expand Down Expand Up @@ -294,7 +293,7 @@ def _handle_force_color_no_color(

final_reporters: list[reporters.BaseReporter] = []

for idx, rep in enumerate(reporters):
for rep in reporters:
if (
no_color
and isinstance(rep, ColorizedTextReporter)
Expand Down

0 comments on commit 8d8f988

Please sign in to comment.