search

tanakh / cargo-atcoder

Cargo subcommand for AtCoder
BSD 3-Clause "New" or "Revised" License
397 stars 36 forks source link

Some command require users to login for even finished contests #31

Closed qryxip closed 4 years ago

qryxip commented 4 years ago

現在newやtest等のコマンドは終了後のコンテストに対してもログインを要求します。

例えばtest時のこれは一旦見て404なら"Please login first"で良いはずです。 https://github.com/tanakh/cargo-atcoder/blob/f93acbb42aa446faf14aac08a764e4a70a01e5bf/src/atcoder.rs#L378-L381

不要なログインを要求しないことで #30 のようなテストが追加できます。 repository owner(=tanakhさん)のクレデンシャルをこんな感じで使うという方法も無くはないですが。

tanakh commented 4 years ago

なるほど、どうせsubmitするならそれらのコマンドがログインを要求しても使い勝手としては変わらないのかなと思っていましたが、テストなどを考えるなら、利便性に繋がりますかね。ありがとうございます。