Skip to content

Conversation

@tokai3594
Copy link
Owner

仕様

Rails8.1に対応させる

変更点

  • Rails8.1にバージョンアップ
  • Rails8.1までのテストフロー作成

Copy link

@patorash patorash left a comment

Choose a reason for hiding this comment

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

レビューしました。

spec.add_development_dependency "base64"
spec.add_development_dependency "drb"
spec.add_development_dependency "concurrent-ruby", '1.3.4'
spec.add_development_dependency "minitest", "= 5.26.1"
Copy link

Choose a reason for hiding this comment

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

minitestのバージョンを固定している理由はありますか?
なければ、~> 5.26のようにしておくのがいいのではないかと思います。

@@ -12,7 +12,18 @@ jobs:
strategy:
matrix:
ruby: ['3.0', '3.1', '3.2', '3.3', '3.4']
Copy link

Choose a reason for hiding this comment

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

Rubyのバージョン4.0が出ているから、そちらも対応したほうがいいのでは?と思います。

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