ktaroabobon / abobon-jr

0 stars 1 forks source link

linter, fomatter, typecheck導入 #11

Closed ktaroabobon closed 5 months ago

ktaroabobon commented 6 months ago

やること

参考資料

ktaroabobon commented 6 months ago

Goに変更するため再調査

ktaroabobon commented 6 months ago

Goでは以下で行う

ktaroabobon commented 6 months ago

一方でgolangci-lintでもgofmtは動きそう

https://golangci-lint.run/usage/configuration/#linters-configuration

ktaroabobon commented 6 months ago

ローカルでフォーマット上書きのためにmake fmtコマンドを使用する それ以外はmake lintでgolangci-lintを動かすこととする

ktaroabobon commented 6 months ago

CIに関しては、golangci-lintを使用する 公式docに載っているものをそのまま使う

https://github.com/golangci/golangci-lint-action

ktaroabobon commented 6 months ago

makeコマンドを実行すると以下のエラーが出るのでちょっと調査

WARN[0000] /abobon-jr/docker-compose.yaml: version is obsolete

ktaroabobon commented 6 months ago

docker-compose.yamlのversionは記述しなくてよくなったらしいので削除

https://forums.docker.com/t/docker-compose-yml-version-is-obsolete/141313

ktaroabobon commented 6 months ago

公式の記述において他のCIとファイルを分けた方が良いという記述があるため、ファイル名を変更する

We recommend running this action in a job separate from other jobs (go test, etc.) because different jobs run in parallel.

https://github.com/golangci/golangci-lint-action?tab=readme-ov-file#how-to-use