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

Aberta
#9,214 2 comentários 1 reação 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
48/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Estagnada
Stack de tecnologia
python
Domínio
tooling

Direção de pesquisa

Comece em mypy/config_parser.py por volta da linha 85 e reproduza o problema com o mypy.ini mostrado e a variável de ambiente A. Verifique a ordem de divisão e expansão dos caminhos; está concluído quando um valor como $A se expande para /foo e /bar, e ambos os caminhos são reconhecidos.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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()))
Linguagem predominante
Python
Estrelas
20.6k
Forks
3.3k
Merge médio
1d 18h
PRs com merge (30d)
54

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de python/mypy

Todas as issues de python/mypy

Issues semelhantes

Mais issues de Python

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.