canonical / canonical/layer-basic

Makefile does not define tox lint tests correctly

Ouverte
#87 2 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
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

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.