Skip to content

Commit

Permalink
Remove rack from the gemfile
Browse files Browse the repository at this point in the history
It was initially added in #987 but the reasoning is lost on me.

Rails 6.1 and 7.0 only work with Rack 2 which causes dependabot update PRs to break CI when it bumps to Rack 3

Rails head will also switch to be Rack 3 only at some point. Just let bundler resolve to what works.

GoodJob does have to do some version-specific rack handling but I don't think this needs to be part of the matrix
  • Loading branch information
Earlopain committed Oct 10, 2024
1 parent e820719 commit 8a8f186
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 2 deletions.
1 change: 0 additions & 1 deletion Gemfile
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ gemspec

gem 'activerecord-jdbcpostgresql-adapter', platforms: [:jruby]
gem 'pg', platforms: [:mri, :mingw, :x64_mingw]
gem 'rack', '~> 2.2'

rails_versions = {
"6.1" => { github: "rails/rails", branch: "6-1-stable" }, # https://github.com/bensheldon/good_job/issues/1280
Expand Down
1 change: 0 additions & 1 deletion Gemfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -517,7 +517,6 @@ DEPENDENCIES
memory_profiler
pg
puma
rack (~> 2.2)
rack-mini-profiler
rails
rbtrace
Expand Down

0 comments on commit 8a8f186

Please sign in to comment.