Skip to content

Commit 83612ce

Browse files
Silence graphql_definition deprecation warning before suporting GQL 2.0 (#217)
* Silence graphql_definition deprecation warning before suporting GQL 2.0 * Only silence deprecation warning if graphql >= 1.13.1 The »silence_deprecation_warning»-param was introduced with v1.13.1 See rmosolgo/graphql-ruby@3d1c24a * Add support for graphql 1.13 This is a cherry-picked version of 5642ca9 excluding the rails-7 related changes. * Temp skip ruby 2.5 rails 6 GQL 1.13 Cherry-picked 502c555 * Skip ruby 2.2 rails 6 GQL 1.13 GQL 1.13 reuires ruby 2.4 Co-authored-by: Mario Celi <mcelicalderon@gmail.com>
1 parent 8a7d2df commit 83612ce

7 files changed

Lines changed: 31 additions & 33 deletions

File tree

.circleci/config.yml

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,33 +66,51 @@ workflows:
6666
- gemfiles/rails5.2_graphql1.11.gemfile
6767
- gemfiles/rails6.0_graphql1.11.gemfile
6868
- gemfiles/rails6.0_graphql1.12.gemfile
69+
- gemfiles/rails6.0_graphql1.13.gemfile
6970
- gemfiles/rails6.1_graphql1.11.gemfile
7071
- gemfiles/rails6.1_graphql1.12.gemfile
72+
- gemfiles/rails6.1_graphql1.13.gemfile
7173
exclude:
7274
- ruby-version: '2.2'
7375
gemfile: gemfiles/rails6.0_graphql1.11.gemfile
7476
- ruby-version: '2.2'
7577
gemfile: gemfiles/rails6.0_graphql1.12.gemfile
78+
- ruby-version: '2.2'
79+
gemfile: gemfiles/rails6.0_graphql1.13.gemfile
7680
- ruby-version: '2.2'
7781
gemfile: gemfiles/rails6.1_graphql1.11.gemfile
7882
- ruby-version: '2.2'
7983
gemfile: gemfiles/rails6.1_graphql1.12.gemfile
84+
- ruby-version: '2.2'
85+
gemfile: gemfiles/rails6.1_graphql1.13.gemfile
8086
- ruby-version: '2.3'
8187
gemfile: gemfiles/rails6.0_graphql1.11.gemfile
8288
- ruby-version: '2.3'
8389
gemfile: gemfiles/rails6.0_graphql1.12.gemfile
90+
- ruby-version: '2.3'
91+
gemfile: gemfiles/rails6.0_graphql1.13.gemfile
8492
- ruby-version: '2.3'
8593
gemfile: gemfiles/rails6.1_graphql1.11.gemfile
8694
- ruby-version: '2.3'
8795
gemfile: gemfiles/rails6.1_graphql1.12.gemfile
96+
- ruby-version: '2.3'
97+
gemfile: gemfiles/rails6.1_graphql1.13.gemfile
8898
- ruby-version: '2.4'
8999
gemfile: gemfiles/rails6.0_graphql1.11.gemfile
90100
- ruby-version: '2.4'
91101
gemfile: gemfiles/rails6.0_graphql1.12.gemfile
102+
- ruby-version: '2.4'
103+
gemfile: gemfiles/rails6.0_graphql1.13.gemfile
92104
- ruby-version: '2.4'
93105
gemfile: gemfiles/rails6.1_graphql1.11.gemfile
94106
- ruby-version: '2.4'
95107
gemfile: gemfiles/rails6.1_graphql1.12.gemfile
108+
- ruby-version: '2.4'
109+
gemfile: gemfiles/rails6.1_graphql1.13.gemfile
110+
- ruby-version: '2.5'
111+
gemfile: gemfiles/rails6.0_graphql1.13.gemfile
112+
- ruby-version: '2.5'
113+
gemfile: gemfiles/rails6.1_graphql1.13.gemfile
96114
- ruby-version: '2.7'
97115
gemfile: gemfiles/rails4.2_graphql1.8.gemfile
98116
- ruby-version: '3.0'

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ README.md.*
2424
# rvm config files
2525
.ruby-version
2626
.ruby-gemset
27+
.tool-versions
2728

2829
.env
2930
/spec/tmp/config/routes.rb

Appraisals

Lines changed: 5 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -82,20 +82,6 @@ appraise 'rails5.2-graphql1.12' do
8282
gem 'rspec-rails', '< 4.0'
8383
end
8484

85-
appraise 'rails6.0-graphql1.8' do
86-
gem 'sqlite3', '~> 1.4'
87-
gem 'devise', '>= 4.7'
88-
gem 'rails', github: 'rails/rails', branch: '6-0-stable'
89-
gem 'graphql', '~> 1.8.0'
90-
end
91-
92-
appraise 'rails6.0-graphql1.9' do
93-
gem 'sqlite3', '~> 1.4'
94-
gem 'devise', '>= 4.7'
95-
gem 'rails', github: 'rails/rails', branch: '6-0-stable'
96-
gem 'graphql', '~> 1.9.0'
97-
end
98-
9985
appraise 'rails6.0-graphql1.10' do
10086
gem 'sqlite3', '~> 1.4'
10187
gem 'devise', '>= 4.7'
@@ -117,11 +103,11 @@ appraise 'rails6.0-graphql1.12' do
117103
gem 'graphql', '~> 1.12.0'
118104
end
119105

120-
appraise 'rails6.1-graphql1.9' do
106+
appraise 'rails6.0-graphql1.13' do
121107
gem 'sqlite3', '~> 1.4'
122108
gem 'devise', '>= 4.7'
123-
gem 'rails', github: 'rails/rails', branch: '6-1-stable'
124-
gem 'graphql', '~> 1.9.0'
109+
gem 'rails', github: 'rails/rails', branch: '6-0-stable'
110+
gem 'graphql', '~> 1.13.0'
125111
end
126112

127113
appraise 'rails6.1-graphql1.10' do
@@ -145,10 +131,9 @@ appraise 'rails6.1-graphql1.12' do
145131
gem 'graphql', '~> 1.12.0'
146132
end
147133

148-
appraise 'rails6.1-graphql_edge' do
134+
appraise 'rails6.1-graphql1.13' do
149135
gem 'sqlite3', '~> 1.4'
150-
gem 'devise_token_auth', github: 'lynndylanhurley/devise_token_auth', branch: 'master'
151136
gem 'devise', '>= 4.7'
152137
gem 'rails', github: 'rails/rails', branch: '6-1-stable'
153-
gem 'graphql', github: 'rmosolgo/graphql-ruby', branch: 'master'
138+
gem 'graphql', '~> 1.13.0'
154139
end
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,6 @@ source "https://rubygems.org"
55
gem "sqlite3", "~> 1.4"
66
gem "devise", ">= 4.7"
77
gem "rails", github: "rails/rails", branch: "6-0-stable"
8-
gem "graphql", "~> 1.8.0"
8+
gem "graphql", "~> 1.13.0"
99

1010
gemspec path: "../"

gemfiles/rails6.0_graphql1.9.gemfile

Lines changed: 0 additions & 10 deletions
This file was deleted.
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,6 @@ source "https://rubygems.org"
55
gem "sqlite3", "~> 1.4"
66
gem "devise", ">= 4.7"
77
gem "rails", github: "rails/rails", branch: "6-1-stable"
8-
gem "graphql", "~> 1.9.0"
8+
gem "graphql", "~> 1.13.0"
99

1010
gemspec path: "../"

lib/graphql_devise/schema_plugin.rb

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,11 @@ def authenticate_option(field, trace_data)
113113
auth_required = if trace_data[:context]
114114
field.metadata[:authenticate]
115115
else
116-
field.graphql_definition.metadata[:authenticate]
116+
if Gem::Version.new(GraphQL::VERSION) >= Gem::Version.new('1.13.1')
117+
field.graphql_definition(silence_deprecation_warning: true).metadata[:authenticate]
118+
else
119+
field.graphql_definition.metadata[:authenticate]
120+
end
117121
end
118122

119123
auth_required.nil? ? @authenticate_default : auth_required

0 commit comments

Comments
 (0)