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

Add Rails 6.1 tests to matrix strategy #72

Merged
merged 7 commits into from
Nov 22, 2023
Merged

Conversation

schoblaska
Copy link
Contributor

I added a Rails 6.1 test fixture app alongside the existing Rails 7.0 one, extended the require_test_rails_application spec helper to accept the Rails version as an argument which defaults to the RAILS_VERSION environment variable. I then modified the matrix test strategy in .github/workflows/ci.yml to run the test suite against RAILS_VERSION=6.1 and RAILS_VERSION=7.0 for each of the different Ruby versions we test against.

If the RAILS_VERSION env var is not set, Rails 7.0 is used as the default.

@schoblaska schoblaska enabled auto-merge (squash) November 22, 2023 16:13
@schoblaska schoblaska merged commit 81bc3ff into main Nov 22, 2023
@schoblaska schoblaska deleted the jms/rails-6-specs branch November 22, 2023 16:50
@@ -3,5 +3,5 @@
#
# Examples:
#
# movies = Movie.create([{ name: 'Star Wars' }, { name: 'Lord of the Rings' }])
# Character.create(name: 'Luke', movie: movies.first)
# movies = Movie.create([{ name: "Star Wars" }, { name: "Lord of the Rings" }])
Copy link
Contributor

Choose a reason for hiding this comment

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

🤨

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