Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Run CI tests against Rails 7.2 #394

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ohbarye
Copy link

@ohbarye ohbarye commented Aug 22, 2024

Hi, based on Rails 7.2 release, this pull request makes CI run tests against Rails 7.2.

I thought I should add Ruby 3.3 to the matrix as well, but I found #383 so this pull request just add Rails 7.2.

@@ -3,5 +3,6 @@
Gemfile.lock
coverage
gemfiles/*.lock
gemfiles/vendor
Copy link
Author

Choose a reason for hiding this comment

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

📝 [chore] I needed to run BUNDLE_GEMFILE=gemfiles/rails_7.2.gemfile bundle to debug test failures but it creates files under gemfiles/vendor. So I decided to let it ignore them.

Copy link
Contributor

@rud rud left a comment

Choose a reason for hiding this comment

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

Excellent idea

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants