search

madwind / flexget_qbittorrent_mod

flexget qbittorrent删种、辅种 自动签到 插件
MIT License
552 stars 117 forks source link

Add type hints & refactor #88

Closed vivodi closed 2 years ago

vivodi commented 2 years ago

添加 from __future__ import annotaions 可以保证兼容 Python 3.9

vivodi commented 2 years ago

好的，我会参考您的意见进行修改或与您讨论。

vivodi commented 2 years ago

你指出的问题我大部分都进行了修改。没有进行修改的问题也对其原因回复了你。

vivodi commented 2 years ago

总结一下：

  1. PEP 484 要求在默认参数为 None 时仍显示地注明参数类型包含 None
  2. 将默认参数的类型推断为参数类型不是标准行为，因此即使有默认参数，依然需要标注参数类型，否则参数类型会被推断为 Any。见 https://github.com/python/mypy/issues/3090#issuecomment-290691282
  3. 在使用赋值表达式简化代码时还要兼顾代码的易读性。用赋值表达式简化代码的例子可以参考 PEP 572
vivodi commented 2 years ago

你说的问题我都进行了修改或者进行了回复