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

BCDice ライブラリを v2 から v3 に更新 #193

Merged
merged 6 commits into from
Jan 9, 2021

Conversation

koi-chan
Copy link
Member

@koi-chan koi-chan commented Jan 5, 2021

gem になった BCDice ライブラリを使うようにしました。
現段階ではまだテストバージョンですが、問題なく動作していると思います。

@koi-chan koi-chan self-assigned this Jan 5, 2021
@koi-chan
Copy link
Member Author

koi-chan commented Jan 7, 2021

テストを修正しました。

現在は RC 版の BCDice を使っているため、バージョン番号のテストが失敗しています。
正式版が出たら正式版しか使わない予定ですので、この失敗は放置するつもりです。

@ochaochaocha3
Copy link
Member

バージョン番号のテストの条件はもっと緩くしてもよさそうですね。v2とは違ってsemverになったので、大雑把に /\ABCDice Version: \d+\.\d+\d+/ くらいのマッチでいいかもしれません。

Copy link
Member

@ochaochaocha3 ochaochaocha3 left a comment

Choose a reason for hiding this comment

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

変更点は問題ないと思います。SW 2.0のテストも通っていますし、大丈夫そうです。新しいBCDiceはライブラリとして十分に使えそうですね。

@koi-chan
Copy link
Member Author

koi-chan commented Jan 8, 2021

バージョン番号のテストの条件はもっと緩くしてもよさそうですね。v2とは違ってsemverになったので、大雑把に /\ABCDice Version: \d+\.\d+\d+/ くらいのマッチでいいかもしれません。

緩くし、テストが通るようにしました。

@ochaochaocha3 ochaochaocha3 merged commit eb51173 into master Jan 9, 2021
@ochaochaocha3 ochaochaocha3 deleted the use-bcdice-v3 branch January 9, 2021 01:46
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