canonical / canonical/layer-basic

Makefile does not define tox lint tests correctly

オープン
#87 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
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

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。