some tests with approved stubs have the wrong error
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 20.6k
- Fork
- 3.3k
- Metriche di merge delle PR
- Metriche PR in attesa
Descrizione
for example, the testLimitLegacyStubErrorVolume test:
https://github.com/python/mypy/blob/bd530392cdbe444c63cc21820ad7b9748ea9c816/test-data/unit/check-modules.test#L3015-L3018
since certifi is an approved stub, it should instead suggest installing them, like it does here:
>mypy -c "import certifi"
<string>:1: error: Library stubs not installed for "certifi" (or incompatible with Python 3.10) [import]
<string>:1: note: Hint: "python3 -m pip install types-certifi"
<string>:1: note: (or run "mypy --install-types" to install all missing stub packages)
<string>:1: note: See https://mypy.readthedocs.io/en/stable/running_mypy.html#missing-imports
Found 1 error in 1 file (checked 1 source file)
but for some reason, these tests (and also the playground) don't do that
Guida per i contributori
Apri la guida per i contributori
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.
Direzione di ricerca
Inizia da test-data/unit/check-modules.test intorno a testLimitLegacyStubErrorVolume e confronta il relativo errore atteso con l’esempio certifi nell’issue. Leggi la voce approved-stub in mypy/stubinfo.py e riproduci il comportamento con il comando mypy fornito. Il lavoro è completato quando gli approved stubs suggeriscono in modo coerente di installare il loro types package nei test interessati e il comportamento del playground è quello previsto.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- tooling
- Tipo di issue
- Bug
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100