astral-sh/ruff

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

Open

#17 699 ouverte le 29 avr. 2025

Voir sur GitHub
 (7 commentaires) (0 réactions) (0 assignés)Rust (47 527 stars) (2 088 forks)batch import
bugfixeshelp wantedrule

Description

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:

Guide contributeur

Stack technique
pythonrust
Domaine
toolingdeveloper experience
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Pythonpathlibruff rulesfile descriptors
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65
Direction de recherche
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.