budougumi0617 / go_todo_app

書籍「詳解Go言語Webアプリケーション開発」のサンプルコードリポジトリです。
https://www.c-r.com/book/detail/1462
MIT License
197 stars 36 forks source link

golangci-lintの設定でdeprecatedの警告が出るようです。 #70

Closed nnabeyang closed 1 year ago

nnabeyang commented 1 year ago

ローカルでもlintのチェックした方が良いなと思い、実行してみたところ、次のような警告が出ました。

WARN [runner] The linter 'deadcode' is deprecated (since v1.49.0) due to: The owner seems to have abandoned the linter.  Replaced by unused.

警告に書かれているように.golangci.ymldeadcodeunusedにすると解消しました。

nnabeyang commented 1 year ago

unusedはdeadcodeと同等ではないようで、unusedの場合はexportedな関数であれば呼び出しが無くても警告は出ないようです。

budougumi0617 commented 1 year ago

サンプルコードの設定ファイルを修正しておきました、ありがとうございました!

nnabeyang commented 1 year ago

@budougumi0617 まだ以下の箇所にdeadcodeが残っているのですが、これはそのままで問題無いでしょうか?

https://github.com/budougumi0617/go_todo_app/blob/main/_chapter15/section59/.golangci.yml#L13 https://github.com/budougumi0617/go_todo_app/blob/main/_chapter16/section60/.golangci.yml#L13 https://github.com/budougumi0617/go_todo_app/blob/main/_chapter16/section61/.golangci.yml#L13 https://github.com/budougumi0617/go_todo_app/blob/main/_chapter16/section62/.golangci.yml#L13 https://github.com/budougumi0617/go_todo_app/blob/main/_chapter16/section63/.golangci.yml#L13 https://github.com/budougumi0617/go_todo_app/blob/main/_chapter16/section64/.golangci.yml#L13 https://github.com/budougumi0617/go_todo_app/blob/main/_chapter17/section68/.golangci.yml#L13 https://github.com/budougumi0617/go_todo_app/blob/main/_chapter17/section70/.golangci.yml#L13 https://github.com/budougumi0617/go_todo_app/blob/main/_chapter18/section74/.golangci.yml#L13 https://github.com/budougumi0617/go_todo_app/blob/main/_chapter19/section75/.golangci.yml#L13 https://github.com/budougumi0617/go_todo_app/blob/main/_chapter19/section78/.golangci.yml#L13 https://github.com/budougumi0617/go_todo_app/blob/main/_chapter19/section79/.golangci.yml#L13

budougumi0617 commented 1 year ago

以下の理由より参考コードの修正はしないつもりです 🙏

nnabeyang commented 1 year ago

@budougumi0617 了解しました。ご対応いただきまして、ありがとうございました。