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

mypyの設定を有効化する #263

Closed
qwerty2501 opened this issue Jan 2, 2022 · 5 comments · Fixed by #959
Closed

mypyの設定を有効化する #263

qwerty2501 opened this issue Jan 2, 2022 · 5 comments · Fixed by #959

Comments

@qwerty2501
Copy link
Contributor

qwerty2501 commented Jan 2, 2022

内容

TODOの掘り起こしです
#30 で行われたlint設定でmypyがとりあえず無効化されてたのでエラー部分も直して有効化する

実現方法

pysenの設定で mypyの有効化及び "strict" modeに変更する

@Hiroshiba
Copy link
Member

mypyは結構理不尽と言うか、pythonの良さを消している部分もあるので、適用は賛否両論ありそうです。
今はどうか知りませんが、例えばdef __init__の返り値にNoneを指定しないとエラーみたいなのがあった気がします。

が、たしかに型の補助はあったほうがコーディングも楽になるし、ミスにも気付けるので良いかもと感じました。
todoとまではいきませんが、一旦検討しても良さそうだと思います。

@qwerty2501
Copy link
Contributor Author

やるにしてもまずはmypyの有効化のみのほうがよさそうですね。

@Hiroshiba
Copy link
Member

こちらのタスクですが、mypyテストがあればよりミスに気づきやすくなると思うのでちょっと優先度上げた方が良い気がしました。
例えば↓のプルリクで修正されたミスはmypyで指摘されました。

個人的にはno-untyped-defを無効化しつつ、他の若干直感的じゃない設定を無効化して導入したいです。

@sabonerune
Copy link
Contributor

mypyの設定はpysenが支配しているようでsetup.cfgを書き換えても反映されないようです。
現在のentryが最も緩い設定のようでこれ以上緩めるならプラグインを書く必要があるかもしれません。

https://github.com/pfnet/pysen?tab=readme-ov-file#frequently-asked-questions

Q. How do I change specific settings for linter X?
A. We prioritize convention over configuration. However you can always create your own plugin. See: Create a plugin to customize pysen

もしくはpysenの設定を無効にしたまま独自に設定をして直接mypyを実行するとか?

@Hiroshiba
Copy link
Member

@sabonerune コメントありがとうございます!!!

な~るほどです。
調べてみた感じpysenをメンテナンスしているpfnetのリポジトリでもpysenが使われていて、見た感じ-> Noneもちゃんと書かれてました。
だったらまあ僕たちも従って良い気がしました!!!

ということで、特に設定なしのまま導入する方向で・・・!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants