[next major] Do not shadow Python standard-library module

Abierto

#246 abierto el 20/1/2025 2 comentarios 0 reacciones 1 asignado Ver en GitHub

@benoit74 ya está trabajando en esto.

Desde el 20/1/2025.

Evaluación

Este issue todavía no se ha evaluado.

Descripción

enhancement

Ruff / Flake8 has a new rule A005: https://docs.astral.sh/ruff/rules/stdlib-module-shadowing/

It is recommended to not shadow Python standard-library modules.

Currently, we have 5 issues:

src/zimscraperlib/html.py:1:1: A005 Module `html` shadows a Python standard-library module
src/zimscraperlib/logging.py:1:1: A005 Module `logging` shadows a Python standard-library module
src/zimscraperlib/rewriting/html.py:1:1: A005 Module `html` shadows a Python standard-library module
src/zimscraperlib/types.py:1:1: A005 Module `types` shadows a Python standard-library module
src/zimscraperlib/typing.py:1:1: A005 Module `typing` shadows a Python standard-library module

For now, I've disabled the rule A005 in https://github.com/openzim/python-scraperlib/pull/245 but we need to make a finer decision.

I think that logging module is shadowing "on-purpose" and might be allowed. Others could be considered for renaming. But I don't have very good suggestions of names, so I'm not 100% convinced, if we end-up with bad names, I'm not sure it really ease maintenance at all.

Lenguaje dominante: Python
Estrellas: 31
Forks: 27
Merge medio: 3 d 7 h
PR fusionados (30 d): 2

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

Lee el issue completo y luego la guía de contribución del proyecto.
Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
Haz un fork del repositorio y trabaja en una rama.
Abre un pull request que haga referencia al número del issue.

Más de openzim/python-scraperlib

HTML rewriting: also rewrite `poster` attribute Abierto

bug

Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100

openzim/python-scraperlib#339 · Abierto el 16/9/2026
[next major] Remove deprecated 'convert' parameter for optimize_image function inside optimization.py Abierto

enhancement

Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100

openzim/python-scraperlib#292 · Abierto el 11/4/2026
URL normalisation: URL-encoded `#` makes its way into the path, but is lost when obtaining the URI of that entry Abierto

bug

Dificultad 4/5 3-5 días Aptitud para principiantes 35/100

openzim/python-scraperlib#341 · Abierto el 17/9/2026
URL normalisation: do not rewrite consecutive slashes `//` as a single slash `/` Abierto

bug

Dificultad 3/5 1-2 días Aptitud para principiantes 45/100

openzim/python-scraperlib#340 · Abierto el 17/9/2026
Add fuzzy rule to rewrite URLs of lesbases.anct.gouv.fr Abierto

bug

openzim/python-scraperlib#334 · 1 asignado · Abierto el 21/8/2026

Todos los issues de openzim/python-scraperlib

Issues similares

slack-gif-creator: validate_gif() multiplies the last frame's duration by the frame count, so reported duration/fps are wrong (1.0s GIF reported as 1.8s) Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

anthropics/skills#1811 · 1 comentario · Abierto el 23/9/2026
App fails to start with gradio 6 (allowed by `gradio>=5.49.1`) Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

speaches-ai/speaches#678 · Abierto el 23/9/2026
ModuleNotFoundError: No module named 'mcp.server.fastmcp'. Abierto

bug

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

datalayer/mcp-compose#42 · Abierto el 23/9/2026
import spacy fails with ModuleNotFoundError: No module named 'click' Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

conda-forge/spacy-feedstock#177 · Abierto el 23/9/2026
humaneval: extract_function_body corrupts a fenced body-only reply, so the same correct code scores INCORRECT fenced and CORRECT unfenced Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100

UKGovernmentBEIS/inspect_evals#2523 · Abierto el 23/9/2026

Más issues de Python

Un resumen breve de issues de GitHub para principiantes.