ryo-n commented 4 years ago

826 対応です。

kmyk commented 4 years ago

CI が落ちていますが、pylint が過剰反応してるだけのようです。以下に too-many-function-args を追加してしまってください :bow:

https://github.com/online-judge-tools/oj/blob/45d70b23d56ea5eda0318e154639218e6bd12d51/setup.cfg#L40-L71

ryo-n commented 4 years ago

一部修正しましたがまだmypyで落ちていそうですね。修正方法調べておきます。現状のままでも一応json.dumpできていそうですが。。

onlinejudge_command/subcommand/download.py:101: error: Argument 2 to "dump" has incompatible type "BinaryIO"; expected "IO[str]" Found 1 error in 1 file (checked 39 source files)

kmyk commented 4 years ago

一部修正しましたがまだmypyで落ちていそうですね。修正方法調べておきます。現状のままでも一応json.dumpできていそうですが。。

onlinejudge_command/subcommand/download.py:101: error: Argument 2 to "dump" has incompatible type "BinaryIO"; expected "IO[str]" Found 1 error in 1 file (checked 39 source files)

そのエラーはファイルをテキストモードで開くかバイナリモードで開くかの差についてのものに見えます。参考: https://docs.python.org/ja/3/tutorial/inputoutput.html#:~:text=%E9%80%9A%E5%B8%B8%E3%80%81%E3%83%95%E3%82%A1%E3%82%A4%E3%83%AB%E3%81%AF%E3%83%86%E3%82%AD%E3%82%B9%E3%83%88%E3%83%A2%E3%83%BC%E3%83%89,%E3%83%90%E3%82%A4%E3%83%8A%E3%83%AA%E3%83%A2%E3%83%BC%E3%83%89%E3%82%92%E5%88%A9%E7%94%A8%E3%81%99%E3%82%8B%E3%81%B9%E3%81%8D%E3%81%A7%E3%81%99%E3%80%82

kmyk commented 4 years ago

よさそうだったのでマージしました。なお、このプルリクは主にテストの修正のみでユーザの実行には影響しないので、すぐには本体のバージョンを上げません。そのうち他のプルリクと一緒に出荷されると思うので待っていてください。

online-judge-tools / oj

Add log file option #828

826 対応です。