#3 shellcheck（静的解析）と shfmt（lint）の実装

[KEINOS]

Issue #3 の実装です。

とりあえず、このドラフト PR で以下を実装します 。

実装（？）してみました。Merge されないと GitHub Actions の動画確認できないっぽい ... Fork 先でも何かできませんでした。

すること

• 再現・開発用の Dockerfile （現在 Alpine Linux）
• shellcheck および shfmt の実行スクリプト（./.github/run-lint.sh）
• 初回 shellceck および shfmt の修正
• GitHub Action による shellceck および shfmt のテスト

しないこと

• 静的解析と lint の修正を除く既存スクリプトのリファクタリング（関数化など）
• 各スクリプトファイルにある関数の単体テスト
• 新たな機能追加

[KEINOS]

lint と静的解析の修正とは言え、やはり、これだけのファイルの修正があると単体テストがホスィ ...
動作テストも面倒なので怖い
テスト書きたくなるけど、でもそれは別 PR ... 我慢 ... 我慢 ...

[KEINOS]

とりえあず、現状の状態のレビューお願いします m(_ _)m

[yoshi389111]

試しに GitHubActions を動かしてみたところ（自分のところの master あてにPRを送った。マージはしていない）、以下のようなエラーが出ました。

取り急ぎ、ご報告まで

---

【追記 2021-05-26 23:56】

go mod download を削除して実行したところ、正常に実行できているようにみえます。

[KEINOS]

やはり、どこかにマージしないと動かせないのかしら。gh とか act も一旦マージされないとローカルで実行できなさそうでした。

go mod download を削除

あー！！！go.mod なんてないから、そら動かないですよね！

[KEINOS]

@yoshi389111

削除してみました。お手数ですがお手すきに確認お願いします。

大丈夫そうだったら Squash & Merge して、テストの作成に入りたいと思います。

[yoshi389111]

確認してみました。前回のエラーは出ていません（正常に終了しています）が、気になる点が何点か

適当にシェルを修正して2回目のpushをすると、キャッシュが効いていないように思います。

• actions/cache@v2 の step に id が付いていないから？
• id をつけると２回目の実行でエラー（shfmtがインストールされていない）。とりあえずキャッシュ対象を ~/go にすると動くようです。

archive を shellcheck に引っ掛かるような修正をしてpushしましたが、エラーになっていません。
run-lint.sh がうまく動いていないようです。

• shellcheckのループ while [ ! "${1:+none}" ]; do を while [ -n "${1:+none}" ]; do に修正
• shfmt のループ while [ ! "$1" ]; do を while [ -n "${1:+none}" ]; do に修正
• shellcheck で失敗しても result が設定されていないようです。shfmtと同じように result を定義/設定/返却するとOK

また、以下でエラーが出ています

• check コマンドが shfmt に引っ掛かる（余計なスペースが入っているとのこと）

[KEINOS]

2回目のpushをすると、キャッシュが効いていない

ご確認ありがとうございます！ご指摘の通り、キャッシュが効いてもビルドされたバイナリがキャッシュに含まれていないことが原因でした。

コミット 597a646 でキャッシュを消しました。キャッシュが必要なくらいの重さであったら再考したいと思います。

archive を shellcheck に引っ掛かるような修正をしてpushしましたが、エラーになっていません。

false positive が出ていることを確認しました。今日の夕方に修正いたします。

[KEINOS]

@yoshi389111

正常が異常なのだ問題の修正と、shfmt のエラーを修正しました！
お手すきにチェックお願いします。

[yoshi389111]

確認しました。問題ないと思います

LGTM

[yoshi389111]

少しだけ気になったのは、ワークフローの定義 .github/workflows/shellcheck_linux.yml は、インデントがスペース２個になっています。

（jsonファイルはインデント 4 になっているようなので）
yml,yamlを分離して 2 にするか、あるいは shellcheck_linux.yml のインデントをスペース4個にすべきではないでしょうか？

修正はあとでもよい気がしますが

[KEINOS]

確かに .editorconfig で定義しているのにそう・・でないのは気持ち悪いですね。気づきませんでした。さすがです。

分離して、ポカよけでフォーマッタを入れてみました。