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

DesperateRunTRPG と ジキルとハイドとグリトグラのダイスボット、テストコードを追加。 #362

Merged
merged 9 commits into from
Jan 4, 2021

Conversation

saronpasu
Copy link
Contributor

神武来(かむらい)さんによる同人ルール、「DesperateRunTRPG」「ジキルとハイドとグリトグラ」の
2作品のダイスボットを寄稿します。
同作者さんが以前に、どどんとふ向けとして作成、公開されていたダイスボットをベースとして
現在のBCDiceに対応する形で書き直しました。

ルール本文はテキストファイルで、旧どどんとふ向けのダイスボットも作者サイトで配布されています。
配布場所

2作品共にテストコードもそれぞれ含めています。
確認よろしくおねがいします。

@codecov
Copy link

codecov bot commented Jan 3, 2021

Codecov Report

Merging #362 (8503424) into master (ce24046) will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #362      +/-   ##
==========================================
+ Coverage   94.46%   94.47%   +0.01%     
==========================================
  Files         269      271       +2     
  Lines       18417    18471      +54     
==========================================
+ Hits        17397    17451      +54     
  Misses       1020     1020              
Impacted Files Coverage Δ
lib/bcdice/game_system.rb 100.00% <100.00%> (ø)
lib/bcdice/game_system/DesperateRun.rb 100.00% <100.00%> (ø)
lib/bcdice/game_system/JekyllAndHyde.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 ce24046...8503424. Read the comment docs.

@ysakasin
Copy link
Member

ysakasin commented Jan 3, 2021

@saronpasu 寄稿ありがとうございます確認します。次回から複数のゲームシステムを寄稿していただく際には、ゲームシステムごとにPull Requestを分割するようにお願いします。

@GenKuzumochi
Copy link
Contributor

配布はフリーですが、著作権は放棄しておりません。
有償配布、製作者自称の配布はご遠慮願います。

ライセンス互換がないので、ベースにしているのなら作者さんに伺ったほうが良さそうですかね

@ysakasin
Copy link
Member

ysakasin commented Jan 3, 2021

@GenKuzumochi そうですね。指摘ありがとうございます。

@saronpasu 確認お願いします。

@saronpasu
Copy link
Contributor Author

saronpasu commented Jan 3, 2021

次回から複数のゲームシステムを寄稿していただく際には、ゲームシステムごとにPull Requestを分割するようにお願いします。

すみません。次は分割して個別に送ります。

ライセンス互換がないので、ベースにしているのなら作者さんに伺ったほうが良さそうですかね

作者の神武来さんにはTwitterのDMにて確認、許諾をいただいています。

@ysakasin ysakasin added the new dicebot 新システムの対応 label Jan 4, 2021
@ysakasin
Copy link
Member

ysakasin commented Jan 4, 2021

@saronpasu @sortType がVer2基準でも不正な値になっているのですが、下記のどれが期待する挙動でしょうか

@sortType = 0 # ソート設定(1 = 足し算ダイスでソート有, 2 = バラバラロール(Bコマンド)でソート有, 3 = 1と2両方ソート有)

https://github.com/bcdice/BCDice/blob/v2/src/diceBot/DiceBot.rb#L70

@ysakasin
Copy link
Member

ysakasin commented Jan 4, 2021

下記変更しました

  • IDとクラス名JekyllandHyde -> JekyllAndHyde
  • 名前を"Desperate Run" -> "Desperate Run TRPG"
    • ルールテキストに後者で書かれていたため
  • SORT_KEYの修正
  • 全角英数記号を半角に
  • 区切り の前後の空白を全角から半角に
  • 難易度決定 の後に区切り を入れる

@saronpasu
Copy link
Contributor Author

@sortType がVer2基準でも不正な値になっているのですが、下記のどれが期待する挙動でしょうか

神武来さんの元ファイルでは 1 が指定されていました。
こちらの書き直しがミスです。すみません。

@ysakasin
Copy link
Member

ysakasin commented Jan 4, 2021

@saronpasu sortTypeの確認ありがとうございます。

@ysakasin ysakasin merged commit 128c9da into bcdice:master Jan 4, 2021
@ysakasin
Copy link
Member

ysakasin commented Jan 4, 2021

寄稿ありがとうございます! 取り込みました

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new dicebot 新システムの対応
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants