Skip to content

refactor: ruff ルール A を適用 #1642

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

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

Conversation

tarepan
Copy link
Contributor

@tarepan tarepan commented Apr 23, 2025

内容

ruff ルール A を適用するリファクタリングを提案します。

ルール A は「built-in と同名を使う」に関するルールであり、一般論としてレビューでよく指摘される内容です。これらルールを適用することで現行コードの安全性を向上させ、かつ将来的なレビュー負荷が低減できると考えます。

関連 Issue

無し

@tarepan tarepan requested a review from a team as a code owner April 23, 2025 20:21
@tarepan tarepan requested review from Hiroshiba and removed request for a team April 23, 2025 20:21
@Hiroshiba
Copy link
Member

すみません、ちょっとなぜlinterを適用するのかの方針を先に決めさせていただきたいです!!
とにかく増やしていくのが正解ではないと思っていて、目的をもって適用していきたいなと。

例えばこのPRだとlicenseという変数名がなぜか使えず開発体験が下がるデメリットがありますが、メリットは「インタラクティブモードのときだけ使えるらしいlicnese()という関数が使えなくならない」だけで、どっちのほうが良いのか判断しづらいなと。
方針・目的が先に決まっていれば今後も機械的に決められるので、先にそちらを決めておくのはどうでしょうか?

@tarepan
Copy link
Contributor Author

tarepan commented Apr 24, 2025

なぜlinterを適用するのかの方針を先に決め

👍️
#1646 を建てました。こちらでリンター・フォーマッターに関する広い議論をおこない、その結果を受けて本 PR を進行します。
#1646 での議論完了まで本 PR は draft 化します。

@tarepan tarepan marked this pull request as draft April 24, 2025 08:40
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