Treat relative paths in config file relative to config file, not cwd
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 20.6k
- Forks
- 3.3k
- Métricas de merge de PR
- Métricas de PR pendientes
Descripción
When using mypy_path = src it should be treated relative to the config file (if any), not the current working directory.
This also applies to the default of cache_dir (.cache_dir), which currently gets created in the current working directory always.
I.e. a detected/found (project) config file could be treated as the root for options with relative defaults also.
As for mypy_path this could be achieved in the type callback, which then however would need the config file / root as a new argument:
https://github.com/python/mypy/blob/84126ab9985844d8cda3809688034af895fdaf7c/mypy/config_parser.py#L75
As for global user config files, it should treat relative files therein relative to it, but it should not be used as a root for default settings like cache_dir.
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza en mypy/config_parser.py, alrededor del type callback de la línea 75, y sigue cómo la raíz de configuración del proyecto detectada se pasa al procesamiento de opciones. Verifica que los valores relativos de mypy_path y los valores predeterminados de cache_dir usen el directorio de configuración del proyecto, mientras que las rutas relativas en los archivos de configuración de usuario globales usen el directorio de ese archivo sin convertirlo en la raíz de la configuración predeterminada.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- tooling
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100