Go 1.19 にバージョンアップ

[kobayashi-m42]

issueURL

https://github.com/nekochans/lgtm-cat-api/issues/39

Doneの定義

https://github.com/nekochans/lgtm-cat-api/issues/39 の完了条件を満たしていること

変更点概要

• Go 1.19 にバージョンアップ
• Go のバージョンアップにより Panic since update to v1.43.0 (load embedded ruleguard rules: rules/rules.go:13: can't load fmt) golangci/golangci-lint#2374 の影響で golangci-lint がエラーになったので、golangci-lint も最新バージョンにアップデート
  • バージョンアップによりエラーが追加されたので対応
  • golangci-lint で指摘されたエラーを回避するために ReadHeaderTimeout を設定
  • タイムアウトの時間は、Lambda 自体のタイムアウトが29秒となっており、それより短い時間の10秒を適当に指定
  • 参考：https://christina04.hatenablog.com/entry/server-request-timeout

また、Issueとは関係がないが下記の変更を行なった。

• Go 1.17 よりテスト実行時に shuffle オプションを指定できるようになったので、テスト時にオプションを指定するように変更
  • このオプションを指定することでテストがランダムに実行され、前に書いたテストの実行結果に依存する状態を検知し、回避することが可能となる
  • 詳細については、下記の参考記事を参照
    • Go 1.17のtesting新機能 | フューチャー技術ブログ shuffleオプションについて
    • O’Reilly 実用Go言語』(p.302 13.6.2 テストの順序依存の排除)

補足

https://github.com/nekochans/lgtm-cat-api/issues/9 はこのPRで対応不要となるので対応無しでクローズする予定。

[codecov]

Codecov Report

Base: 86.81% // Head: 86.81% // No change to project coverage 👍

Coverage data is based on head (7324f5e) compared to base (ecfbc7a).
Patch has no changes to coverable lines.

Additional details and impacted files

@@           Coverage Diff           @@
##             main      #64   +/-   ##
=======================================
  Coverage   86.81%   86.81%           
=======================================
  Files           2        2           
  Lines          91       91           
=======================================
  Hits           79       79           
  Misses         10       10           
  Partials        2        2           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[keitakn]

ありがとう🐱

shuffle オプションいいね、デフォルトで指定しておいても良さそう👌