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

[BloodCrusade][BloodMoon] '両システム間で共用されていたが本来は細部の異なる' いくつかのテーブルを、個別に持つように #388

Merged
merged 2 commits into from
Feb 26, 2021

Conversation

ViVi-shark
Copy link
Contributor

@ViVi-shark ViVi-shark commented Feb 25, 2021

背景

ゲームシステムに関する背景

  • 『ブラッドムーン』は『ブラッド・クルセイド』を前身システム(のひとつ)とする
  • 『ブラッドムーン』のルールおよび一部のテーブルは『ブラッド・クルセイド』とほぼ同様の内容である

BCDice 上の実装に関する背景

  • ”ほぼ同様”のテーブルについて、実装が再利用されていた

(このPRで解決される予定の)課題

 それらのテーブルのうち一部は、ゲームシステム上、”ほぼ同様”ではあっても完全に同一ではない

  • 『ブラッドムーン』『ブラッド・クルセイド』で改名された用語がある(【感情】→【テンション】など)
  • 狂気のデータにも異なる点がある

 上記は一例。
 具体的な該当箇所のすべては e03f4dd#diff-2c743aefb650d2f294db4d3db23e42843547a1747f7f3fffd50d17bb45ab8c86 を参照。

対応方針

 差異のあるテーブルは使いまわしをやめて個別に持つようにする。

@codecov
Copy link

codecov bot commented Feb 25, 2021

Codecov Report

Merging #388 (2c31426) into master (bdcb6b2) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #388   +/-   ##
=======================================
  Coverage   94.48%   94.48%           
=======================================
  Files         272      272           
  Lines       18514    18514           
=======================================
  Hits        17493    17493           
  Misses       1021     1021           
Impacted Files Coverage Δ
lib/bcdice/game_system/BloodCrusade.rb 100.00% <ø> (ø)
lib/bcdice/game_system/BloodMoon.rb 100.00% <100.00%> (ø)

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 bdcb6b2...e03f4dd. Read the comment docs.

@ysakasin ysakasin merged commit b23aa3c into bcdice:master Feb 26, 2021
@ViVi-shark ViVi-shark deleted the features/fix_blood-moon branch March 28, 2021 10:17
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