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.

Sign up for GitHub

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

Jump to bottom

[Arknights Fan TRPG] アップデート #700

Closed
wants to merge 44 commits into from
Closed

[Arknights Fan TRPG] アップデート #700

wants to merge 44 commits into from

Conversation

Ayase00
Copy link
Contributor

@Ayase00 Ayase00 commented May 6, 2024

・成功、失敗とクリティカル、エラーの優先度をルール準拠に変更。
・ヘルプメッセージの誤りを修正。
・ココフォリアのルーム変数に対応するため、役職名の後ろに0がつくと動作せず、1がつくと動作するように整備。

Ayase and others added 30 commits January 1, 2024 20:08
first_commit
b4855b8
2nd commit
f2d7196 
>D, B -> AD, AB
>Increased test casees.
> Enabled orthographical variants. eg, AN100<=10, AN<=10.
3rd commit
4ee09a7 
> Syndrome tables are added.
> Dice result is sorted.
@Ayase00
Refined.
e0b958c
@Ayase00
4th commit
58af5ca 
> Old functions (recentry unused) were removed.
> Dice table was changed from array to hash.
@Ayase00
4.2 commits
e14fe52 
> Remained old function was removed.
@NOBUTOKA
[ArknightsTRPG] Add example comments on command persers.
0c0a0e0
@Ayase00
[ArknightsTRPG]
598ee92 
> Refined by rubocop.
@Ayase00
[ArknightsTRPG]
8b7a9aa 
> Reduce 'when' sections in func eval_game_system_specific_command().
@Ayase00
[ArknightsTRPG]
85e84bf 
> Refined by rubocop.
@Ayase00
[ArknightsTRPG]
79d8929 
> Reduce 'when' sections in func eval_game_system_specific_command().
@Ayase00
[ArknightsTRPG]
a1beada 
> Comments by N.K was merged.
@Ayase00
[ArknightsTRPG]
ea1824c 
> Refined by Rubocop.
@NOBUTOKA
[ArknightsTRPG] > Add testcaces for (\d*) patterns.
dbc866a
@Ayase00
[ArknightsTPRG]
29a7f12 
Merge branch 'develop'
> Refined with Rubocop.
> Reduced 'when' sections.
> Add testcaces for (\d*) patterns.
@ysakasin
[ArknightsFan] SORT_KEY を修正
938015e
@ysakasin
[ArknightsFan] コマンドのパース位置を変更
b2098be
@ysakasin
[ArknightsFan] refactor
812b4de
@ysakasin
[ArknightsFan] refactor
8a8dfb2
@opparco @NOBUTOKA
NightWizard3rd: Fix help message
0354b48
@Faceless192x @ysakasin @NOBUTOKA
『フルフェイス』に対応(bcdice#668)
e118f29 
* 判定を実装。Rubocop通過済み

* バースト時の表示を変更した

* [FullFace] コマンドを完全一致にする

* [FullFace] Array#count の使い方を変更

---------

Co-authored-by: SAKATA Sinji <ysakasin@gmail.com>
@Faceless192x @ysakasin @NOBUTOKA
『アニマラス』を追加 (bcdice#669)
bb3d81d 
* 判定一式を実装、Rubocop通過済み

* オフィシャルに合わせてリネーム

* ヘルプの誤りを修正

* [AniMalus] コマンドを完全一致にする

* [AniMalus] Array#count でブロックが不要な点を修正

* [AniMalus] 不要な #to_i を削除

---------

Co-authored-by: SAKATA Sinji <ysakasin@gmail.com>
@NOBUTOKA
[VampireTheMasquerade5th] ダイスプールよりHungerダイスが大きい時に全てがHungerダイスになるように変更
cc1c5ce
@NOBUTOKA
[WerewolfTheApocalypse5th] ダイスプールよりRageダイスが大きい時に全てがRageダ イスになるように変更
44fbfa3
@ysakasin @NOBUTOKA
Support Ruby 3.3
c75c12e
@ysakasin @NOBUTOKA
Drop Ruby 2.6 support
f8375fb
@ysakasin @NOBUTOKA
Update rubocop
8961b01
@ysakasin @NOBUTOKA
Update development dependencies
0e69715
@ysakasin @NOBUTOKA
Cop
eb57788
@hikoyamamayo @NOBUTOKA
「パストフューチャーパラドックス」を追加 (bcdice#673)
3420265
@raa0121
Copy link
Contributor

raa0121 commented May 6, 2024

途中でマージしたりしているため、入れたい差分に対して、コミットの状況がおかしくなっています。
必要な差分だけ保存して、再度作業をやり直せないでしょうか。
おすすめは fork しなおして、今の lib/bcdice/game_system/ArknightsFan.rb と test/data/ArknightsFan.toml のみを上書きして、再度PRを出していただくことです。
ご検討よろしくお願いします。

@ysakasin おせっかいかも知れませんが、コミット整理を提案させていただきました。

ysakasin
ysakasin requested changes May 9, 2024
View reviewed changes
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.

古いコミットが残ってしまっているので、コミットを整理するようお願いします

@Ayase00
Copy link
Contributor Author

Ayase00 commented May 16, 2024

対応遅れてしまい申し訳ありません。
コミット整理承知いたしました。
Pull request の前にフォーク元(BcDice本体)の変更をマージして取り込んでおく処理は止めておいたほうが良いでしょうか？

@raa0121
Copy link
Contributor

raa0121 commented May 16, 2024

こちらですが、前回の Arknights Fan TRPG の取り込みが、squash merge となっており、コミットがまとめられてます。#666 f1db21f
なので、手元とBCDice 本体のツリー情報に差分があり、過去のコミットがこのPRに混じっているという状況になっています。
したがって、BCDice の master から新しくブランチを切り直して、そこに今回の差分を乗せるという形が良いと思います。
こちらの方針については、Arknights Fan TRPG のコードの共同開発者の @NOBUTOKA さんにもDiscord で伝えてあります。
よろしくお願いします。

@Ayase00
Copy link
Contributor Author

Ayase00 commented May 16, 2024

ありがとうございます。discordの方確認致しました。２度お手間をおかけする形となってしまい申し訳ありませんでした。
対応したく思いますので、少々お時間頂けますと幸いです。

@Ayase00 Ayase00 marked this pull request as draft May 16, 2024 13:33
@Ayase00 Ayase00 closed this Aug 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ysakasin ysakasin ysakasin requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@Ayase00 @raa0121 @ysakasin @NOBUTOKA @opparco @Faceless192x @hikoyamamayo