Skip to content

Commit

Permalink
Remove upper bound dependency limits from gemspec (#711)
Browse files Browse the repository at this point in the history
* Remove upper bound limit from our dependencies

All that was needed to support Rails 7 in this gem was to update the
gemspec. This means that even in the most likely release to break
anything, a major release, we didn't needed to worry about breaking this
gem.

This upper bound is just creating unnecessary work for the
maintainers of this gem, which would need to release it without any code
change, and the users, which need to open issues asking for new releases.

Let's allow our users to try any version of our dependencies and report
real issues if they find them, and save everyone's time just removing
the upper bound.

* Add changelog entry

---------

Co-authored-by: Javier Julio <[email protected]>
  • Loading branch information
rafaelfranca and javierjulio authored Oct 6, 2023
1 parent 20a65ac commit 204e34a
Show file tree
Hide file tree
Showing 6 changed files with 17 additions and 20 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

## Master (unreleased)

* Remove upper bound dependency limits from gemspec.
* Allow using Rails 7.1.

## Version 1.13.1
Expand Down
8 changes: 4 additions & 4 deletions Gemfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@ PATH
remote: .
specs:
inherited_resources (1.13.1)
actionpack (>= 6.0, < 7.2)
has_scope (~> 0.6)
railties (>= 6.0, < 7.2)
responders (>= 2, < 4)
actionpack (>= 6.0)
has_scope (>= 0.6)
railties (>= 6.0)
responders (>= 2)

GEM
remote: https://rubygems.org/
Expand Down
8 changes: 4 additions & 4 deletions gemfiles/rails_61/Gemfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@ PATH
remote: ../..
specs:
inherited_resources (1.13.1)
actionpack (>= 6.0, < 7.2)
has_scope (~> 0.6)
railties (>= 6.0, < 7.2)
responders (>= 2, < 4)
actionpack (>= 6.0)
has_scope (>= 0.6)
railties (>= 6.0)
responders (>= 2)

GEM
remote: https://rubygems.org/
Expand Down
8 changes: 4 additions & 4 deletions gemfiles/rails_70/Gemfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@ PATH
remote: ../..
specs:
inherited_resources (1.13.1)
actionpack (>= 6.0, < 7.2)
has_scope (~> 0.6)
railties (>= 6.0, < 7.2)
responders (>= 2, < 4)
actionpack (>= 6.0)
has_scope (>= 0.6)
railties (>= 6.0)
responders (>= 2)

GEM
remote: https://rubygems.org/
Expand Down
8 changes: 4 additions & 4 deletions inherited_resources.gemspec
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,8 @@ Gem::Specification.new do |s|

s.required_ruby_version = '>= 2.7'

s.add_dependency("responders", ">= 2", "< 4")
s.add_dependency("actionpack", ">= 6.0", "< 7.2")
s.add_dependency("railties", ">= 6.0", "< 7.2")
s.add_dependency("has_scope", "~> 0.6")
s.add_dependency("responders", ">= 2")
s.add_dependency("actionpack", ">= 6.0")
s.add_dependency("railties", ">= 6.0")
s.add_dependency("has_scope", ">= 0.6")
end
4 changes: 0 additions & 4 deletions test/tasks/gemspec_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -11,10 +11,6 @@ def teardown
File.delete("inherited_resources-#{InheritedResources::VERSION}.gem")
end

def test_has_no_warnings
refute_includes @build[1], "WARNING"
end

def test_succeeds
assert_equal true, @build[2].success?
end
Expand Down

0 comments on commit 204e34a

Please sign in to comment.