search

icpc-jag / rime

Rime: Automation Tool for Programming Contest Organizers
MIT License
45 stars 28 forks source link

Flake8, fix #9 #22

Closed not522 closed 7 years ago

not522 commented 7 years ago

とりあえず.flake8で指定したエラー以外を修正しました。

TODO

hiroshi-cl commented 7 years ago

乙です ここまで入れたらtravisとかでtestやlint回したいですね

not522 commented 7 years ago

そうですね、CIで回せるようにしたいですね。

どこまで修正を入れたらmergeするかですが、E1xxがインデント周り・E501が1行の文字数制限なので、このあたりは後回しにしてmergeしてしまってもいいかなと思います。 他の3つはなるべく修正してからmergeしたいと思っています。

エラーコード一覧： https://pycodestyle.readthedocs.io/en/latest/intro.html#error-codes http://flake8.pycqa.org/en/latest/user/error-codes.html

not522 commented 7 years ago

インデント幅が2であることに起因するエラー以外は修正しました。 インデント幅の修正は作業が膨大すぎるので、このプルリクではE111,E114,E121は無視することにします。

hiroshi-cl commented 7 years ago

LGTM 基本的にflake8のいうことに従ったわけですし幾つかのレポジトリで動かしてみて問題なければよいでしょう

hiroshi-cl commented 7 years ago

マージボタン押してもいいですかね？

not522 commented 7 years ago

はい、大丈夫です