Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

依存関係の更新 #48

Merged
merged 5 commits into from
Sep 20, 2023
Merged

依存関係の更新 #48

merged 5 commits into from
Sep 20, 2023

Conversation

takanakahiko
Copy link
Member

Why

  • セキュリティの向上
    • 依存が古く Dependabot の警告が出ている

How

  • Update module github.com/urfave/cli to v1.22.14 #39 で導入された github.com/urfave/cli の新しいバージョンへの移行
  • Update module gopkg.in/yaml.v2 to v3 #45 で導入された gopkg.in/yaml の新しいバージョンへの移行
    • gopkg.in/yaml.v3 に移行した関係上、 yaml 内で日時っぽい値が渡された時に time.Time にパースされるようになった。json の値を比較する関係上、日時の比較は文字列でやるべきなので、日時型が渡された場合はエラーとする。
  • GitHub Actions 内の Go のバージョンのアップデート

注意!

日時型のエラーハンドリングを追加したため、次のリリース以降は日時を扱いたい場合はダブルクォーテーションで囲って文字列扱いする必要がある。(リリースノートに書いた方が良い)

@takanakahiko
Copy link
Member Author

@mizzy レビューお願いしたいです 🙏

Copy link
Member

@mizzy mizzy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMです:+1:

@takanakahiko takanakahiko merged commit f304087 into master Sep 20, 2023
3 checks passed
@takanakahiko takanakahiko deleted the update-deps branch September 20, 2023 02:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants