astral-sh/ruff

`PTH*`: Incorrect suggestion to use `Pathlib` when using file descriptors

Open

#17.699 geöffnet am 29. Apr. 2025

Auf GitHub ansehen
 (7 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (47.527 Stars) (2.088 Forks)batch import
bugfixeshelp wantedrule

Beschreibung

Multiple pathlib rules recommend the use of pathlib.Path over their os equivalent even when they're used with a bytes string or directory descriptor which aren't supported by pathlib.

One such example is

import os
os.listdir(1)

$ ruff --version
ruff 0.11.7
$ ruff check --isolated --select PTH208 t.py
t.py:2:1: PTH208 Use `pathlib.Path.iterdir()` instead.
  |
1 | import os
2 | os.listdir(1)
  | ^^^^^^^^^^ PTH208
  |

Found 1 error.

Past instances of the same or similar error:

Contributor Guide

Tech Stack
pythonrust
Domain
toolingdeveloper experience
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
active
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Pythonpathlibruff rulesfile descriptors
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65
Research-Richtung
Investigate the implementation of PTH208 in the ruff codebase, likely in `crates/ruff linter/src/rules/pathlib/`. Check how the rule currently determines when to suggest `pathlib.Path.iterdir()`. The issue indicates that the rule should skip cases where the argument is a bytes string or a file descriptor (integer). Review past issues #7620 and #12871 for prior discussion and potential solutions. Implement a condition to suppress the rule when the argument is not a string path.