-
Notifications
You must be signed in to change notification settings - Fork 194
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
MAINT: FastAPIのパラメータを全てAnnotated
に移行する
#1183
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
👍️ FastAPI の機能を活用し型アノテーションとして API を定義しています。よりモダンな記法なので長期メンテ性が向上しそうですね、good work!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!
flake8-bugbearのB008エラーは、仮引数のデフォルト値にオブジェクトを指定した場合に、インスタンスが関数の呼び出しごとに作られず、関数が読み込まれた時に作られたものが使い回されてしまうという、Pythonのわかりにくい仕様について回避するように指摘するものという理解です。
今回の修正点はFastAPIの特殊な書き方の部分なので、この仕様が実際に問題になることはないと思いますが、Python 3.9で導入されたtyping.Annotated
に置き換えて警告を解消することで、文脈を知らない人にも理解しやすいコードになったと思います!
内容
を
の記法に統一します。
その他
# noqa: B008
が消せたAPIは変化していない。