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

[VampireTheMasquerade5th] Hungerダイスを内数指定できるコマンドを追加 #615

Merged
merged 3 commits into from
Apr 8, 2023

Conversation

FredGreenfield
Copy link
Contributor

@FredGreenfield FredGreenfield commented Mar 27, 2023

Vampire:The Masquerade5th(以下V5)にHungerダイスを内数指定できるコマンド、VMIを追加した。

【追加の背景】
V5では、Hunger分だけ判定ダイスプールがHungerダイスに置き換えられる。
これまでは使用するユーザがダイスプールから累積したHungerを引いて、コマンドを打っていた。
e.g.) ダイスプール9、Hungerダイス3の時のコマンド
  VMF6+3

【追加の内容】
Hungerダイス分を引いたダイスプールをユーザが算出する必要がないよう、以下のような内数指定できるコマンドを追加した。
これにより、キャラクターの現状をそのまま入力して判定を行うことができるようになる。

[コマンド書式]
(難易度)VMI(ダイスプール)H(Hungerダイス数

e.g.)ダイスプール9、Hungerダイス3の時のコマンド
  VMI9H3

FredGreenfield and others added 2 commits March 26, 2023 21:47
@codecov
Copy link

codecov bot commented Mar 27, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (f11256a) 95.45% compared to head (bdf20a5) 95.46%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #615   +/-   ##
=======================================
  Coverage   95.45%   95.46%           
=======================================
  Files         333      333           
  Lines       18623    18635   +12     
  Branches     4825     4828    +3     
=======================================
+ Hits        17777    17790   +13     
+ Misses        846      845    -1     
Impacted Files Coverage Δ
lib/bcdice/game_system/VampireTheMasquerade5th.rb 100.00% <100.00%> (+1.44%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@ysakasin ysakasin merged commit 9229b72 into bcdice:master Apr 8, 2023
@ysakasin
Copy link
Member

ysakasin commented Apr 8, 2023

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

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