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

[CastleInGray] 『灰色城綺譚』 CASTLE IN GRAY The Haunted Palace #495

Merged
merged 4 commits into from
Aug 23, 2021

Conversation

reismannnr2
Copy link
Contributor

ゲームシステム

灰色城綺譚
CASTLE IN GRAY The Haunted Palace

発行: かりかりうめ

関連サイト

かりかりうめ
公式ショップ

コマンド内容

    ■ 色占い (BnWm)
    n: 黒
    m: 白
    n, m は1~12の異なる整数

    例) B12W7
    例) B5W12

    ■ 悪意の渦による占い (MALn)
    n: 悪意の渦
    n は1~12の整数

    ■ その他
    ・感情表 ET
    ・暗示表(黒) BIT
    ・暗示表(白) WIT
  • 判定用コマンドの実装
  • 各種表の追加

ルール自体は非常に簡潔なシステムですが、オンセだと判定が少しややこしいので作成しました。

Copy link
Member

@ysakasin ysakasin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

寄稿ありがとうございます。

DiceTable::Table で追加したコマンドが実行できるか、確認するテストケースを追加してください。

@codecov
Copy link

codecov bot commented Aug 20, 2021

Codecov Report

Merging #495 (3a87897) into master (e4cf96c) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #495   +/-   ##
=======================================
  Coverage   95.58%   95.59%           
=======================================
  Files         296      297    +1     
  Lines       18864    18898   +34     
=======================================
+ Hits        18031    18065   +34     
  Misses        833      833           
Impacted Files Coverage Δ
lib/bcdice/game_system.rb 100.00% <100.00%> (ø)
lib/bcdice/game_system/CastleInGray.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 e4cf96c...3a87897. Read the comment docs.

@reismannnr2 reismannnr2 requested a review from ysakasin August 20, 2021 13:34
@reismannnr2
Copy link
Contributor Author

すみません、追加しておきました。

Copy link
Member

@ysakasin ysakasin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

表テストの追加ありがとうございます。

表のテストですが、全パターンの網羅はしないでください。
各コマンドごとに3パターンほどに抑えてください。

@reismannnr2
Copy link
Contributor Author

迅速な再チェックありがとうございます、両端と中間値のテストに削減しました。

@reismannnr2 reismannnr2 requested a review from ysakasin August 20, 2021 14:12
Copy link
Member

@ysakasin ysakasin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

修正ありがとうございます。
こちらで細かな修正を行ってから取り込もうと思います!

@ysakasin
Copy link
Member

@reismannnr2 BnWm の色を判定する条件式を修正しました。問題ないか確認お願いします。

@reismannnr2
Copy link
Contributor Author

reismannnr2 commented Aug 23, 2021

確認しました! 大丈夫だと思います!
ありがとうございます。

@ysakasin ysakasin merged commit 7edfc0b into bcdice:master Aug 23, 2021
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