Wrong column reported for errors in function sig type comments
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 the following code
def extract_commit_hash(line): # type: (str) -> Optional[str]
...
def extract_commit_hash2(line): # type: (str) -> Optional(str)
...
mypy reports wrong columns:

mypy output:
> mypy --pretty --follow-imports skip --show-column-numbers --show-error-codes wrong-col.py
wrong-col.py:3:10: error: Name 'Optional' is not defined [name-defined]
def extract_commit_hash(line): # type: (str) -> Optional[str]
^
wrong-col.py:3:10: note: Did you forget to import it from "typing"? (Suggestion: "from typing import Optional")
wrong-col.py:6:10: error: Invalid type comment or annotation [valid-type]
def extract_commit_hash2(line): # type: (str) -> Optional(str)
^
wrong-col.py:6:10: note: Suggestion: use Optional[...] instead of Optional(...)
Found 2 errors in 1 file (checked 1 source file)
From what I found the reported column is always 10 even if the code is more indented.

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
Riproduci il report con l’esempio mostrato wrong-col.py e il comando mypy usando --show-column-numbers. Traccia il modo in cui le diagnostiche per i commenti di tipo delle firme delle funzioni calcolano la relativa colonna, quindi verifica che gli errori in funzioni con indentazione diversa indichino la posizione pertinente nel codice sorgente invece di una colonna fissa.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- devtools
- Tipo di issue
- Bug
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 42/100