Closed KEINOS closed 3 years ago
lint と静的解析の修正とは言え、やはり、これだけのファイルの修正があると単体テストがホスィ ... 動作テストも面倒なので怖い テスト書きたくなるけど、でもそれは別 PR ... 我慢 ... 我慢 ...
とりえあず、現状の状態のレビューお願いします m( )m
試しに GitHubActions を動かしてみたところ(自分のところの master あてにPRを送った。マージはしていない)、以下のようなエラーが出ました。
取り急ぎ、ご報告まで
【追記 2021-05-26 23:56】
go mod download
を削除して実行したところ、正常に実行できているようにみえます。
@yoshi389111
削除してみました。お手数ですがお手すきに確認お願いします。
大丈夫そうだったら Squash & Merge して、テストの作成に入りたいと思います。
確認してみました。前回のエラーは出ていません(正常に終了しています)が、気になる点が何点か
適当にシェルを修正して2回目のpushをすると、キャッシュが効いていないように思います。
actions/cache@v2
の step に id が付いていないから?~/go
にすると動くようです。archive を shellcheck に引っ掛かるような修正をしてpushしましたが、エラーになっていません。
run-lint.sh
がうまく動いていないようです。
while [ ! "${1:+none}" ]; do
を while [ -n "${1:+none}" ]; do
に修正while [ ! "$1" ]; do
を while [ -n "${1:+none}" ]; do
に修正result
を定義/設定/返却するとOKまた、以下でエラーが出ています
2回目のpushをすると、キャッシュが効いていない
ご確認ありがとうございます!ご指摘の通り、キャッシュが効いてもビルドされたバイナリがキャッシュに含まれていないことが原因でした。
コミット 597a646 でキャッシュを消しました。キャッシュが必要なくらいの重さであったら再考したいと思います。
archive を shellcheck に引っ掛かるような修正をしてpushしましたが、エラーになっていません。
false positive が出ていることを確認しました。今日の夕方に修正いたします。
@yoshi389111
正常が異常なのだ問題の修正と、shfmt のエラーを修正しました! お手すきにチェックお願いします。
Issue #3 の実装です。
とりあえず、このドラフト PR で以下を実装します 。実装(?)してみました。Merge されないと GitHub Actions の動画確認できないっぽい ... Fork 先でも何かできませんでした。
すること
./.github/run-lint.sh
)しないこと