astral-sh/ruff

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

Open

#17.699 aberto em 29 de abr. de 2025

Ver no GitHub
 (7 comments) (0 reactions) (0 assignees)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:

Guia do colaborador

Pilha de tecnologia
pythonrust
Domain
toolingdeveloper experience
Tipo Issue
bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
active
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Pythonpathlibruff rulesfile descriptors
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65
Direção de pesquisa
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.