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

ArknightsTRPGのダイスボット追加 #666

Merged
merged 16 commits into from
Jan 24, 2024
Merged

Conversation

Ayase00
Copy link
Contributor

@Ayase00 Ayase00 commented Jan 3, 2024

ArknightsTRPG (二次創作) のダイスボット追加リクエストです。

ダイステーブルについて、事情により自作を行いましたが、デフォルトのもので動作可能かつ必要と判断されましたらご指示を頂けますと幸いです。詳細につきましてはArknightsFan.rb line: 208, 209 を参照ください。

Ayase and others added 12 commits January 1, 2024 20:08
>D, B -> AD, AB
>Increased test casees.
> Enabled orthographical variants. eg, AN100<=10, AN<=10.
> Syndrome tables are added.
> Dice result is sorted.
> Old functions (recentry unused) were removed.
> Dice table was changed from array to hash.
> Remained old function was removed.
> Refined by rubocop.
> Reduce 'when' sections in func eval_game_system_specific_command().
> Comments by N.K was merged.
> Refined by Rubocop.
@Ayase00 Ayase00 changed the title ArknightsTRPG (二次創作) のダイスボット追加 ArknightsTRPGのダイスボット追加 Jan 3, 2024
@ysakasin ysakasin self-requested a review January 6, 2024 06:38
Copy link

codecov bot commented Jan 22, 2024

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (c3deac1) 95.63% compared to head (8a8dfb2) 95.67%.
Report is 4 commits behind head on master.

Files Patch % Lines
lib/bcdice/game_system/ArknightsFan.rb 96.73% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #666      +/-   ##
==========================================
+ Coverage   95.63%   95.67%   +0.03%     
==========================================
  Files         346      349       +3     
  Lines       20022    20276     +254     
  Branches     5250     5319      +69     
==========================================
+ Hits        19148    19399     +251     
- Misses        874      877       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ysakasin ysakasin merged commit f1db21f into bcdice:master Jan 24, 2024
8 checks passed
@ysakasin
Copy link
Member

@Ayase00 ありがとうございます。マージしました!

@Ayase00
Copy link
Contributor Author

Ayase00 commented Jan 24, 2024

@Ayase00 ありがとうございます。マージしました!

@ysakasin 対応いただきありがとうございます!

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