canonical / canonical/layer-basic

make lint has redundant behavior

オープン
#76 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Python
スター
10
フォーク
42
PR マージ指標
30日以内にマージされた PR はありません

説明

Bundletester explicitly calls charm-proof, so we really don't need to call it in the makefile during lint.

background info:

having this added to the makefile is redundant, and necessitates me to fork the makefile in my upper layer if I wish to remove this behavior. Charm-tools hasn't been bumped in quite a while and charms leveraging new features will consistently fail when using keywords like:
- terms
- resources

I feel like having lint fail consistently, when proof has already failed, raises red herrings.

Can we remove the proof step from the lint target to stop the redundancy?

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

Makefile の lint ターゲットと、Bundletester による charm-proof の明示的な呼び出しを調査します。現在 lint が proof を重複して実行していること、およびその呼び出しを削除すると Bundletester が proof を担当することを確認します。lint ターゲットが charm-proof を呼び出さなくなり、既存の lint フローがそのまま維持されれば完了です。

索引モデルが issue の本文から書いたものです。

評価

領域
build-system
issue の種類
リファクタリング
難易度
1/5
見積もり時間
1時間未満
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
35/100

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

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