-
Notifications
You must be signed in to change notification settings - Fork 185
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
[NeverCloud]説明追加、誤字修正、コマンドの変更 #539
Conversation
Codecov Report
@@ Coverage Diff @@
## master #539 +/- ##
==========================================
+ Coverage 95.56% 95.57% +0.01%
==========================================
Files 322 323 +1
Lines 18850 18901 +51
==========================================
+ Hits 18014 18065 +51
Misses 836 836
Continue to review full report at Codecov.
|
@spore0814
コマンドの変更は慎重になりたい部分です。どのように誤って呼び出してしまうのか、詳しく教えてください。 |
ご依頼時にいただいた文章によると「記述を途中で打ってしまってコマンド後の文章を出してしまう事があったから」とのことでした。 |
なるほど。事情を理解できました。 そのような誤送信はあらゆるコマンドに発生しうることなので、コマンド名を変える利益よりも、コマンドを変えない利益の方が大きいように思えます。誤送信を防ぐという理屈だと、 他の変更点は特に問題ないです! |
LIKEの変更については、私も懸念を持たれることであろうと思っていましたので、詳細について私立彩音学園の担当者の方から頂いてきました。少し長いのですが、こちらもお読みいただければ幸いです。
|
補足ありがとうございます。 |
|
|
コマンドについてなのですが、話し合いの結果、今後エラッタなどで文章量が変わることを考慮して、やはり |
(変更するかどうか自体についての意見はのべませんが、) 仮に変更するのなら、ルール文の呼び出し系コマンドは、(それが誤操作でないときは)かなり強い意志をもって実行されるはずなので、 |
@spore0814 コマンドの変更を受け入れます。他に変更するコマンド名があれば追加でコミットしてください。 ViVi-sharkさんの提案にある |
テキスト、表を「アルファベット4文字+数字」、表の一覧表示を「アルファベット4文字+数字+L」の形式で統一しました。 |
@spore0814 ありがとうございます。マージしました! |
概要
ネバークラウドTRPGのダイスボットの更新を行いました。
ネバークラウドTRPGの作者である私立彩音学園様のご依頼により作成しております。
内容
ルールの呼び出しがコマンドになっている点の説明を追加。
RoC表の一覧表示が可能である点の説明を変更。
「CHAR2」であるべきところが「CHRA2」になっていたため修正。
「LIKE」コマンドを誤って呼び出してしまうことが多いため、「LIKE1」コマンドに変更
「LIST」コマンドの説明を判定の説明より前に持ってくるよう変更、文面を調整。
その他
テストファイルの末尾に改行を追加しました。
また、このPRが解決しましたら、Result対応をしたいと思っています(ひとつのPRで対応するには大きな変更であると思ったため)。