online-judge-tools / api-client

API client to develop tools for competitive programming
MIT License
64 stars 17 forks source link

Remove toml from requires #167

Closed kzrnm closed 7 months ago

kzrnm commented 11 months ago

yosupo06/library-checker-problems を実行するために toml パッケージをインポートしていますが、最新版では不要になっています。

Latest yosupo06/library-checker-problems doesn't require toml package.

https://github.com/yosupo06/library-checker-problems/pull/801

koba-e964 commented 11 months ago

https://github.com/yosupo06/library-checker-problems/pull/801 で変更されていた README.md の修正、および toml.py の削除をお願いできますか?

kzrnm commented 11 months ago

どういうことでしょうか?

kzrnm commented 11 months ago

わからないこと

何を変更すべきかがわかりません。

https://github.com/yosupo06/library-checker-problems/pull/801 の変更を戻してほしい」ということでしょうか? そうであるならば、なぜ戻す必要があるのかわかりません。

https://github.com/online-judge-tools/api-client になんらかの変更を加えてほしい」ということでしょうか? そうであるならば、api-client は toml.py を持っていないようなので対象のファイルがわかりません。

koba-e964 commented 11 months ago

すみません、https://github.com/yosupo06/library-checker-problems/pull/801 がこのリポジトリーの変更だと勘違いしていました。(病気) CI が落ちているようなので、この PR に起因するものか元々落ちていたのか調べます。今のところは変更していただく箇所はありません。お手数おかけしました。

kzrnm commented 11 months ago

CI についてこちらでも見てみます。

kzrnm commented 11 months ago

@koba-e964 とりあえず Python のバージョンと Codeforces の名前変更は確実にコケるのでそこを修正しました。 https://github.com/online-judge-tools/api-client/pull/168

kzrnm commented 7 months ago

rebase して無事 CI 通りました

koba-e964 commented 7 months ago

ありがとうございます 🙇‍♀️