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

update dependencies #1452

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nebocco
Copy link
Contributor

@nebocco nebocco commented Oct 28, 2023

ref #1126

sqlx のアップデートによりメモリリークが解決したと判明したら 上記 issue が close できるかと思います。

対応したこと

  • sqlx を 0.7 にアップグレードし、コードに必要な修正を加えました。大半は型を明示する必要が生じたことによるものです。
  • ついでに他の crate も最新のものにアップグレードしました。
  • Dockerfile.dev で使用される Rust のバージョンをリリース版と同じものに引き上げました。

対応してないこと

  • chrono.Utc.timestamp() が depricated となり、 chrono.Utc.timestamp_opt() が推奨されています。ただしこの関数の返り値は DateTime ではなく LocalResult<DateTime> であり、剥がし方が分からなかったのでそのままにしてあります。

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.

1 participant