Skip to content

[初音ミクTRPG ココロダンジョン]韓国語対応を追加 & i18n化#855

Merged
ysakasin merged 2 commits intobcdice:masterfrom
MY4k:Miku_kr_MY4k
Apr 8, 2026
Merged

[初音ミクTRPG ココロダンジョン]韓国語対応を追加 & i18n化#855
ysakasin merged 2 commits intobcdice:masterfrom
MY4k:Miku_kr_MY4k

Conversation

@MY4k
Copy link
Copy Markdown
Contributor

@MY4k MY4k commented Mar 1, 2026

初音ミクTRPG ココロダンジョンのダイスボットのi18n化と韓国語対応を追加しました。

translate_tablesにD66Tableが多いため、Metrics/PerceivedComplexityが制限値を超えています [46/37]。
.rubocop_todo.ymlを更新する形でよいでしょうか?他に良い対処法があればぜひ教えていただけると嬉しいです。
いつもありがとうございます。

追記: translate_tablesgeneral_tableskeyword_tablesname_tablesotodama_tablesの4つのメソッドに分割することで解決しました。この方法で問題なければ幸いです。

Copy link
Copy Markdown
Member

@ysakasin ysakasin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

D66Table.from_i18n() があるので、それを使うようにしてください。また、これを使えば複雑度がコードの複雑度が下がるので、transrate_tables を分割せずにすむはずです。

@MY4k
Copy link
Copy Markdown
Contributor Author

MY4k commented Apr 7, 2026

ご指摘ありがとうございます。D66Table.from_i18n()を使用して修正しました。
コミット履歴が少し乱れてしまいましたが、もし問題があればお知らせいただけると幸いです。

@ysakasin ysakasin merged commit 99369d8 into bcdice:master Apr 8, 2026
8 checks passed
@ysakasin
Copy link
Copy Markdown
Member

ysakasin commented Apr 8, 2026

@MY4k 対応ありがとうございます。マージしました。

@MY4k MY4k deleted the Miku_kr_MY4k branch April 8, 2026 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants