[ネオンアンダーレルム] 専用ダイスボット追加#829
Merged
ysakasin merged 9 commits intobcdice:masterfrom Feb 15, 2026
Merged
Conversation
2384ff8 to
3cfba95
Compare
Contributor
Author
|
@ysakasin 申し訳ありません。テストプレイヤーに確認したところ、修正値の位置を末尾指定可能にしてほしいとのことで、追加で仕様とコード追加を行いました。 |
ysakasin
requested changes
Feb 14, 2026
Member
ysakasin
left a comment
There was a problem hiding this comment.
寄稿ありがとうございます。いくつか気になった点があるので、修正をおねがいします。
Comment on lines
+112
to
+114
| # 加減算のみの想定だが、念のため整数以外は弾く | ||
| i = v.to_i | ||
| return nil unless v == i |
Member
There was a problem hiding this comment.
BCDice::Arithmetic.eval() の内部では端数の丸め処理が実施されるため、必ず整数が返却されます。そのため、この判定は不要です。
また、整数かの判定をしたいのであれば、 Numeric#integer? を使うべきです。
Co-authored-by: SAKATA Sinji <ysakasin@gmail.com>
Co-authored-by: SAKATA Sinji <ysakasin@gmail.com>
Co-authored-by: SAKATA Sinji <ysakasin@gmail.com>
Contributor
Author
|
@ysakasin 指摘事項対応いたしました。 Suggestionを取り入れた上で同様のコードを直しています。 |
ysakasin
approved these changes
Feb 15, 2026
Member
|
修正ありがとうございます。マージしました。 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
自作TRPG、ネオン・アンダーレルムの専用ダイスボットを実装いたしました。
レビューとマージのほどよろしくお願いいたします。
ものとしては、
Mb10<=Nのダイスカウントロールに特定の閾値以下のダイス個数をカウントできる仕様です。呼び出しコマンドが
数値式NU(nu許可)です。他システムと重複していたらお知らせください。以上、よろしくお願いします。