canonical / canonical/layer-basic
Makefile does not define tox lint tests correctly
- Lingua principale
- Python
- Stelle
- 10
- Fork
- 42
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
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.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Direzione di ricerca
Inizia nel Makefile e confronta il target lint con gli ambienti tox segnalati e PATH. Esegui bundletester per riprodurre l’errore flake8 mancante, quindi usa il target lint proposto nell’issue e verifica che il comando lint e il passaggio charm proof vengano completati correttamente.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- build-system, testing
- Tipo di issue
- Bug
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Stato di attività
- Ferma
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 52/100