mypy_path option split on [:,] before environment variable expansion

Ouverte
#9,214 2 commentaires 1 réaction 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
48/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
À l'abandon
Stack technique
python
Domaine
tooling

Piste de recherche

Commencez dans mypy/config_parser.py autour de la ligne 85 et reproduisez le problème avec le mypy.ini indiqué et la variable d’environnement A. Vérifiez l’ordre de séparation et d’expansion des chemins ; c’est terminé lorsqu’une valeur telle que $A s’étend à la fois vers /foo et vers /bar, et que les deux chemins sont reconnus.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

feature

Hi,

given an environment variable composed of multiple path, eg export A=/foo:/bar, using the following mypy.ini fails to recognize the two paths foo and bar

[mypy]
mypy_path = $A

This is a consequence of splitting the option on [:,] before expanding environment variables. (https://github.com/python/mypy/blob/master/mypy/config_parser.py#L85)

Would it be sufficient / not harmful to replace the parsing function with the following ?

lambda s: re.split('[,:]', expand_path(s.strip()))
Langage dominant
Python
Étoiles
20.6k
Forks
3.3k
Merge moyen
1 j 18 h
PR mergées (30 j)
54

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de python/mypy

Toutes les issues de python/mypy

Issues similaires

Plus d'issues Python

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.