search

kenkoooo / AtCoderProblems

Extend your AtCoder
https://kenkoooo.com/atcoder/
MIT License
1.39k stars 149 forks source link

Fix language name normalization #1438

Closed roumcha closed 9 months ago

roumcha commented 9 months ago

Fixes #1424, fixes #1436, related to #1431

言語名中のバージョン名や環境名をより正確に除去するための変更と、テストの追加・変更を行いました。

1436 で「議論の余地がありそう」とした変更を、とりあえずすべて含めてあります。

（#1431 と一部競合する形となっており、当該PR作成者様には申し訳ありません。）

kenkoooo commented 9 months ago

なんか CI がエライぶっ壊れてそう

roumcha commented 9 months ago

まず、CIをちゃんと理解せずにPRを出していました、お手間を取らせてしまいすみません！

フロントエンドの方は、TypeScript 4.0 の Labeled Tuple を古いバージョンで使ったため落ちていました。

バックエンドの方は、test_language_count.rs のテストを変更していませんでした。\ こちらは直したとしても、私の理解不足（と、マシンパワー不足）のため、テストの実行が困難です。

一旦closeかdraftさせていただき、目途が立ったら再度提案するか、他の方が修正できるならお任せしたいと思います。

kenkoooo commented 9 months ago

了解です！もらっちゃいますね！