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

Aperta
#9,214 2 commenti 1 reazione 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
48/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Ferma
Stack tecnologico
python
Ambito
tooling

Direzione di ricerca

Inizia in mypy/config_parser.py intorno alla riga 85 e riproduci il problema con il mypy.ini mostrato e la variabile d’ambiente A. Controlla l’ordine di suddivisione ed espansione dei percorsi; il lavoro è completo quando un valore come $A si espande sia in /foo sia in /bar ed entrambi i percorsi vengono riconosciuti.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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()))
Lingua principale
Python
Stelle
20.6k
Fork
3.3k
Merge medio
1g 18h
PR unite (30g)
54

Guida per i contributori

Apri la guida per i contributori

Come iniziare

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

Altre issue di python/mypy

Tutte le issue di python/mypy

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.