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

シノビガミ:新機能追加・誤字の修正 #625

Merged
merged 3 commits into from
Jun 8, 2023

Conversation

NiSLa-NiSlimeTRPG
Copy link
Contributor

・新機能追加
 パニック表、覚醒表、忍法授業シーン表、不良高校シーン表、出物表を追加
 妖術変調対応表で現代のみ・戦国のみのロールに対応
・誤字の修正
 新異形表の誤字の修正(異形態→異霊態)

@codecov
Copy link

codecov bot commented May 14, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (8934a5a) 95.57% compared to head (134fdb5) 95.57%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #625   +/-   ##
=======================================
  Coverage   95.57%   95.57%           
=======================================
  Files         334      334           
  Lines       19153    19153           
  Branches     4974     4974           
=======================================
  Hits        18306    18306           
  Misses        847      847           
Impacted Files Coverage Δ
lib/bcdice/game_system/ShinobiGami.rb 100.00% <ø> (ø)

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

Copy link
Contributor

@GenKuzumochi GenKuzumochi left a comment

Choose a reason for hiding this comment

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

表を追加した場合はテストも必要です。

ちなみに、ターミナルで以下を実行すると、その場でBCDiceのコマンドを試せるので便利です。(変更内容によっては事前にraccは必要なので、一度rakeしてから)

ruby bin/repl.rb

'攻撃系忍法授業シーン表',
'1D6',
[
"接近戦闘学:《刀術》《野戦術》\n
Copy link
Contributor

Choose a reason for hiding this comment

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

Rubyではファイル中の改行も解釈されてしまうので、改行が二重になる&先頭のインデントが出力されてしまいます。

[ShinobiGami]> NCT2
防御系忍法授業シーン表(2) > 警備防衛学:《分身の術》《暗号術》

            【かばう】(基本 p81)/【護衛】(基本 p81)/【陽動】(基本 p102)/好きな忍術

            広大な御斎学園の敷地全体を使った授業。指定されたターゲットの元にすぐさま赴き、授業終了時まで、護衛しなければならない。自分の動きだけでなく、護衛対象に的確な指示を出すこと、その指示を襲撃者に読み取らせないようにする方法も学習できる。

いくつか方法はありますが、素朴にファイル中では改行を入れず、\nで改行するのが一番手っ取り早いかと思います。

"射撃戦闘学:《砲術》《見敵術》\n【必中】(基本 p79)/【文曲】(基本 p103)/【魔弓】(御斎 p33)/好きな器術"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

なるほど…コーディング中に見づらかったからってので勝手に改行してました。申し訳ない
テストファイル追加の件も承知しました。修正しておきます

@NiSLa-NiSlimeTRPG
Copy link
Contributor Author

テストの追加と軽微な誤字修正を行いました。ご確認いただけると幸いです

@GenKuzumochi
Copy link
Contributor

LGTMです。

NCT1等は若干改行多めな気もしますが、そもそもBCDiceがあまり大量の文を出すことを基本とした作りになっていないので、ココフォリアやユドナリウムを想定して見やすそうなら良いと思います。

@NiSLa-NiSlimeTRPG
Copy link
Contributor Author

LGTMです。

NCT1等は若干改行多めな気もしますが、そもそもBCDiceがあまり大量の文を出すことを基本とした作りになっていないので、ココフォリアやユドナリウムを想定して見やすそうなら良いと思います。

基本的にココフォリアを主に想定していますが、Rakeで試してみても可読性の観点から改行の必要性を感じたのであんな感じになってます。

@ysakasin ysakasin merged commit d0c17de into bcdice:master Jun 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.

3 participants