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

修正: pyproject.toml の古いコメントを削除 #1381

Closed
wants to merge 1 commit into from

Conversation

tarepan
Copy link
Contributor

@tarepan tarepan commented Jun 1, 2024

内容

pyproject.toml の古いコメントを削除することを提案します。

関連 Issue

無し

@tarepan tarepan requested a review from a team as a code owner June 1, 2024 15:08
@tarepan tarepan requested review from Hiroshiba and removed request for a team June 1, 2024 15:08
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

・・・・あれ、ちょっとLGTM出した後なのですが違和感が

このメモがないとfastapiから使われてることがわからなくなるので、例えばbenchmarkコードからhttpxへの依存が外れたときに間違えて消してしまう気がしますね!!

より良いコメントの形があるかもですが、少なくともコメントは残しておくべきな気がしてきました・・・!

@Hiroshiba Hiroshiba self-requested a review June 15, 2024 13:46
@tarepan
Copy link
Contributor Author

tarepan commented Jun 15, 2024

benchmarkコードからhttpxへの依存が外れたときに間違えて消してしまう

👍️
妥当な指摘です。
この可能性はすっかり見落としていました。指摘ありがとうございます!

同様の状態にあるライブラリが複数あるため、別 PR にてそれら含めコメントを改良します (#1400)。
本 PR は不適切につき close とします。

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