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

Aperta

#246 aperta il 20/1/2025 2 commenti 0 reazioni 1 assegnatario Vedi su GitHub

@benoit74 ci sta già lavorando.

Dal 20/1/2025.

Valutazione

Questa issue non è ancora stata valutata.

Descrizione

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.

Lingua principale: Python
Stelle: 31
Fork: 27
Merge medio: 3g 7h
PR unite (30g): 2

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Come iniziare

Leggi tutta la issue e poi la guida ai contributi del progetto.
Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
Fai un fork del repository e lavora su un branch.
Apri una pull request che faccia riferimento al numero della issue.

Altre issue di openzim/python-scraperlib

HTML rewriting: also rewrite `poster` attribute Aperta

bug

Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100

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

enhancement

Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100

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

bug

Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100

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

bug

Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100

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

bug

openzim/python-scraperlib#334 · 1 assegnatario · Aperta il 21/8/2026

Tutte le issue di openzim/python-scraperlib

Issue simili

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) Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100

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

Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100

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

bug

Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100

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

Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100

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

Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100

UKGovernmentBEIS/inspect_evals#2523 · Aperta il 23/9/2026

Altre issue su Python