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

[Cthulhu7th] 技能ロールで難易度を指定可能にする #417

Merged
merged 4 commits into from
Apr 16, 2021

Conversation

ysakasin
Copy link
Member

@ysakasin ysakasin commented Apr 15, 2021

・技能ロールの難易度指定 CC(x)<=(目標値)(難易度)
 目標値の後に難易度を指定することで
 成功/失敗/クリティカル/ファンブル を自動判定する。
 難易度の指定:
  r:レギュラー h:ハード e:イクストリーム c:クリティカル
 例)CC<=70r CC1<=60h CC-2<=50e CC2<=99c

@codecov
Copy link

codecov bot commented Apr 15, 2021

Codecov Report

Merging #417 (6bfb12b) into master (745f1bb) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 6bfb12b differs from pull request most recent head f8e941a. Consider uploading reports for the commit f8e941a to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master     #417   +/-   ##
=======================================
  Coverage   95.07%   95.08%           
=======================================
  Files         284      284           
  Lines       18519    18540   +21     
=======================================
+ Hits        17607    17628   +21     
  Misses        912      912           
Impacted Files Coverage Δ
lib/bcdice/game_system/Cthulhu7th.rb 99.28% <100.00%> (+0.12%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 745f1bb...f8e941a. Read the comment docs.

@ysakasin ysakasin marked this pull request as ready for review April 16, 2021 06:12
@ysakasin ysakasin merged commit 4c8488e into master Apr 16, 2021
@ysakasin ysakasin deleted the coc_7th_difficulty_level branch April 16, 2021 06:18
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