Skip to content

Commit 81e4ecf

Browse files
authored
fix: more reliable check of module is included (#1418)
handle gems like GraphQL which override `include?` ``` rake aborted! ArgumentError: wrong number of arguments (given 1, expected 3) gems/graphql-2.0.13/lib/graphql/schema/directive.rb:58:in `include?' ```
1 parent bdce601 commit 81e4ecf

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

lib/tasks/check_upgrade.rake

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ namespace :jsonapi do
99
task :check_upgrade => :environment do
1010
Rails.application.eager_load!
1111

12-
resource_klasses = ObjectSpace.each_object(Class).select { |klass| klass.include?(JSONAPI::ResourceCommon)}
12+
resource_klasses = ObjectSpace.each_object(Class).select { |klass| klass.included_modules.include?(JSONAPI::ResourceCommon)}
1313

1414
puts "Checking #{resource_klasses.count} resources"
1515

0 commit comments

Comments
 (0)