canonical / canonical/layer-basic

Makefile does not define tox lint tests correctly

Abierto
#87 2 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
Python
Estrellas
10
Forks
42
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

When I run bundletester, I consistently get an error about flake8 not being found. I believe that this is because:

1) I evilly do not have flake8 installed in my global Python environment.
2) The Makefile in this layer incorrectly added the py34 and py35 tox virtual environments to its path.

Instead, the Makefile should add the "lint" virtual environment to its path, as below:

```
.PHONY: lint
lint: apt_prereqs
@tox --notest
@PATH=.tox/lint/bin flake8 $(wildcard hooks reactive lib unit_tests tests)
@charm proof
```

I am planning on submitting a patch for this soon -- wanted to record in case I get sidetracked, though.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Empieza en el Makefile y compara el objetivo lint con los entornos tox indicados y PATH. Ejecuta bundletester para reproducir el error de flake8 faltante, luego usa el objetivo lint propuesto en el issue y verifica que el comando lint y el paso de charm proof se completen correctamente.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
python
Área
build-system, testing
Tipo de issue
Error
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
52/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.