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

test: pin sqlite3 dependency to < 2.0 #303

Merged
merged 2 commits into from
Apr 29, 2024
Merged

Conversation

flavorjones
Copy link
Collaborator

Because Rails's sqlite adapter requires sqlite3 ~> 1.4 through v7.1.3.2, tests started failing once the new sqlite3 v2.0 was pulled in:

https://github.com/active-hash/active_hash/actions/runs/8813099625

This has been fixed in Rails 7-1-stable and main but no releases are yet available.

This PR:

  • pins the test dependency to < 2.0 for the relevant Rails versions
  • and also sets fail-fast: false in the github actions pipeline matrix

because the Rails adapter requires `~> 1.4` through v7.1.3.2

we can loosen this for later versions of Rails that contain rails/rails#51592
and any future releases of 7.1 that contain rails/rails#51636
@flavorjones flavorjones requested a review from kbrock April 24, 2024 13:29
Copy link
Collaborator

@kbrock kbrock left a comment

Choose a reason for hiding this comment

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

Lots of gems are also doing this

@kbrock kbrock merged commit 2d642c9 into master Apr 29, 2024
16 checks passed
@kbrock kbrock deleted the flavorjones-dep-sqlite3-lt-2.0 branch April 29, 2024 03:15
@kbrock kbrock self-assigned this Apr 29, 2024
kbrock added a commit to kbrock/active_hash that referenced this pull request Apr 29, 2024
- Support `has_many :through` associations active-hash#296 @flavorjones
- Rails 7.1 support active-hash#291 @y-yagi

- Rails 7.1: fix sqlite3 issue active-hash#303 @flavorjones
- Rails 7.1.3: add missing `has_query_constraints?` active-hash#300 @flavorjones
- `Array#pluck` supports methods active-hash#299 @iberianpig
- Prefer `safe_constantize` over `constantize` active-hash#297 @flavorjones
- Treat `nil` and `blank?` as different values active-hash#295 @kbrock
- Fix `#where` for string keys active-hash#292 @usernam3
kbrock added a commit to kbrock/active_hash that referenced this pull request Apr 29, 2024
- Ruby 3.3 support active-hash#298 @m-nakamura145
- Support `has_many :through` associations active-hash#296 @flavorjones
- Rails 7.1 support active-hash#291 @y-yagi

- Rails 7.1: fix sqlite3 issue active-hash#303 @flavorjones
- Rails 7.1.3: add missing `has_query_constraints?` active-hash#300 @flavorjones
- `Array#pluck` supports methods active-hash#299 @iberianpig
- Prefer `safe_constantize` over `constantize` active-hash#297 @flavorjones
- Treat `nil` and `blank?` as different values active-hash#295 @kbrock
- Fix `#where` for string keys active-hash#292 @usernam3
@kbrock kbrock mentioned this pull request Apr 29, 2024
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