canonical / canonical/layer-basic
Makefile does not define tox lint tests correctly
- Langage dominant
- Python
- Étoiles
- 10
- Forks
- 42
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
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.
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Piste de recherche
Commencez dans le Makefile et comparez la cible lint avec les environnements tox signalés et PATH. Exécutez bundletester pour reproduire l’erreur flake8 manquante, puis utilisez la cible lint proposée dans l’issue et vérifiez que la commande lint et l’étape charm proof se terminent correctement.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- build-system, testing
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 52/100