canonical / canonical/layer-basic
Makefile does not define tox lint tests correctly
- 主要言語
- Python
- スター
- 10
- フォーク
- 42
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
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.
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
調査の方向性
Makefile から始め、lint ターゲットを報告されている tox 環境および PATH と比較します。bundletester を実行して不足している flake8 エラーを再現し、次に issue で提案されている lint ターゲットを使用して、lint コマンドと charm proof ステップが正常に完了することを確認します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- python
- 領域
- build-system, testing
- issue の種類
- バグ
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 52/100