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

NightWizard3rd: Fix help message #667

Merged
merged 1 commit into from
Jan 6, 2024
Merged

NightWizard3rd: Fix help message #667

merged 1 commit into from
Jan 6, 2024

Conversation

opparco
Copy link
Contributor

@opparco opparco commented Jan 6, 2024

ナイトウィザード3rdに関して
#252

ファンブルのとき、常時発動ではない(能動的な)効果による修正値は適用されない。
「ナイトウィザード The 3rd Edition」ではファンブルでも適用される

この記述通り、ルール上、2ndと若干違いがある。
そして、3rdのHELP_MESSAGEは2ndを継承しており、同じになっている。
つまり、3rdの処理は正しいのだけど、説明文が間違っている。
なので、3rd側にHELP_MESSAGEを追加して、訂正した。

ナイトウィザード2ndに関して

aNW+b@x#y$z+c
x : クリティカル値のカンマ区切り(省略時 10)
y : ファンブル値のカンマ区切り(省略時 5)
z : プラーナによる達成値補正のプラーナ消費数(ファンブル時には適用されない)

そして、正規表現上@x#y$zはカンマ区切りのリストで指定できるように書かれているのだけど、

        command.critical_numbers = m[3] ? m[3].split(',').map(&:to_i) : [10]
        command.fumble_numbers = m[4] ? m[4].split(',').map(&:to_i) : [5]
        command.prana = m[5]&.to_i

ここで、command.pranaはリストを取れないので、マッチしないように正規表現を変更した。

Copy link

codecov bot commented Jan 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c3deac1) 95.63% compared to head (89f92c0) 95.63%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #667   +/-   ##
=======================================
  Coverage   95.63%   95.63%           
=======================================
  Files         346      346           
  Lines       20022    20023    +1     
  Branches     5250     5250           
=======================================
+ Hits        19148    19149    +1     
  Misses        874      874           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@opparco
Copy link
Contributor Author

opparco commented Jan 6, 2024

$zの追加はこの時点
#255

@ysakasin ysakasin merged commit 6e646c7 into bcdice:master Jan 6, 2024
8 checks passed
@ysakasin
Copy link
Member

ysakasin commented Jan 6, 2024

ありがとうございます!

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