Require main active_support lib before any of its extension definitions

It seems to be the recommended way with active_support >= 7.0. See https://guides.rubyonrails.org/active_support_core_extensions.html#cherry-picking-a-definition Fix #2205. Signed-off-by: Baptiste Courtois <b.courtois@criteo.com>
ruby-grape · Dec 20, 2021 · f43a8f0 · f43a8f0
1 parent 783fa59
commit f43a8f0
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,7 @@
 
 #### Fixes
 
+* [#2206](https://github.com/ruby-grape/grape/pull/2206): Require main active_support lib before any of its extension definitions - [@annih](https://github.com/Annih)
 * [#2193](https://github.com/ruby-grape/grape/pull/2193): Fixed the broken ruby-head NoMethodError spec - [@Jack12816](https://github.com/Jack12816).
 * [#2192](https://github.com/ruby-grape/grape/pull/2192): Memoize the result of Grape::Middleware::Base#response - [@Jack12816](https://github.com/Jack12816).
 * [#2200](https://github.com/ruby-grape/grape/pull/2200): Add validators module to all validators - [@ericproulx](https://github.com/ericproulx).

diff --git a/lib/grape.rb b/lib/grape.rb
@@ -7,6 +7,7 @@
 require 'rack/auth/basic'
 require 'rack/auth/digest/md5'
 require 'set'
+require 'active_support'
 require 'active_support/version'
 require 'active_support/core_ext/hash/indifferent_access'
 require 'active_support/core_ext/object/blank'