Skip to content

Commit

Permalink
Merge pull request #40 from jrafanie/update_ruby_rails_dependencies
Browse files Browse the repository at this point in the history
Use core minimum but allow 7.0/7.1/7.2 since core sets upper limit (breaking change)
Fryguy authored Oct 8, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
2 parents 6c6328d + a62d109 commit 8ed409a
Showing 3 changed files with 17 additions and 10 deletions.
11 changes: 9 additions & 2 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
@@ -15,10 +15,17 @@ jobs:
- '2.7'
- '3.0'
- '3.1'
- '3.2'
- '3.3'
rails-version:
- '6.0'
- '6.1'
- '7.0'
- '7.1'
- '7.2'
exclude:
- ruby-version: '2.7'
rails-version: '7.2'
- ruby-version: '3.0'
rails-version: '7.2'
services:
postgres:
image: manageiq/postgresql:13
10 changes: 5 additions & 5 deletions Gemfile
Original file line number Diff line number Diff line change
@@ -5,12 +5,12 @@ gemspec

minimum_version =
case ENV['TEST_RAILS_VERSION']
when "6.0"
"~>6.0.4"
when "7.0"
"~>7.0.8"
when "7.2"
"~>7.2.1"
when "7.1"
"~>7.1.4"
else
"~>6.1.4"
"~>7.0.8"
end

gem "activerecord", minimum_version
6 changes: 3 additions & 3 deletions activerecord-id_regions.gemspec
Original file line number Diff line number Diff line change
@@ -7,7 +7,7 @@ Gem::Specification.new do |spec|
spec.name = "activerecord-id_regions"
spec.version = ActiveRecord::IdRegions::VERSION
spec.authors = ["ManageIQ Developers"]

spec.metadata['rubygems_mfa_required'] = 'true'
spec.summary = %q{ActiveRecord extension to allow partitioning ids into regions, for merge replication purposes}
spec.description = %q{ActiveRecord extension to allow partitioning ids into regions, for merge replication purposes}
spec.homepage = "https://github.com/ManageIQ/activerecord-id_regions"
@@ -20,8 +20,8 @@ Gem::Specification.new do |spec|
spec.executables = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
spec.require_paths = ["lib"]

spec.add_dependency "activerecord", ">= 5.0", "<7.1"
spec.add_dependency "activesupport", ">= 5.0", "<7.1"
spec.add_dependency "activerecord", ">= 7.0.8", "<8.0"
spec.add_dependency "activesupport", ">= 7.0.8", "<8.0"
spec.add_dependency "pg"

spec.add_development_dependency "bundler"

0 comments on commit 8ed409a

Please sign in to comment.