Wrong column reported for errors in function sig type comments
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 20.6k
- Forks
- 3.3k
- Métriques de merge des PR
- Métriques de PR en attente
Description
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.

Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Reproduisez le rapport avec l’exemple wrong-col.py présenté et la commande mypy utilisant --show-column-numbers. Suivez la manière dont les diagnostics des commentaires de type des signatures de fonction calculent leur colonne, puis vérifiez que les erreurs dans des fonctions avec une indentation différente indiquent l’emplacement pertinent dans le code source plutôt qu’une colonne fixe.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- devtools
- Type d'issue
- Bug
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 42/100