Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix language name normalization #1438

Merged
merged 3 commits into from
Oct 15, 2023
Merged

Fix language name normalization #1438

merged 3 commits into from
Oct 15, 2023

Conversation

roumcha
Copy link
Contributor

@roumcha roumcha commented Oct 15, 2023

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

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

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

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

@roumcha roumcha marked this pull request as ready for review October 15, 2023 04:13
@kenkoooo
Copy link
Owner

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

@roumcha
Copy link
Contributor Author

roumcha commented Oct 15, 2023

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

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

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

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

@kenkoooo kenkoooo changed the base branch from master to staging October 15, 2023 10:21
@kenkoooo kenkoooo changed the base branch from staging to master October 15, 2023 10:22
@kenkoooo kenkoooo changed the base branch from master to staging October 15, 2023 10:22
@kenkoooo
Copy link
Owner

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

@kenkoooo kenkoooo merged commit bfcd349 into kenkoooo:staging Oct 15, 2023
0 of 2 checks passed
@roumcha roumcha deleted the fix-language-name-normalization branch October 28, 2023 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants