Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for Ruby 3.2 #156

Merged
merged 2 commits into from
Feb 14, 2023
Merged

Add support for Ruby 3.2 #156

merged 2 commits into from
Feb 14, 2023

Conversation

dipth
Copy link
Contributor

@dipth dipth commented Feb 14, 2023

These changes adds support for Ruby 3.2 by bumping the minimum required version of graphql from 1.10.0 to 1.12.18 which fixes a Ruby 3.2 incompatibility related to the use of include in refinements as fixed in this PR:
rmosolgo/graphql-ruby#3674

to add support for Ruby 3.2 related to the use of include in refinements as fixed in this PR:
rmosolgo/graphql-ruby#3674
@dipth dipth mentioned this pull request Feb 14, 2023
@swalkinshaw swalkinshaw merged commit 425d9dc into Shopify:master Feb 14, 2023
@swalkinshaw
Copy link
Contributor

Excellent, thanks @dipth

@shopify-shipit shopify-shipit bot temporarily deployed to rubygems February 14, 2023 16:09 Inactive
@dipth
Copy link
Contributor Author

dipth commented Feb 15, 2023

@swalkinshaw no problem. Any chance you could push an update to rubygems?

@swalkinshaw
Copy link
Contributor

I can, but why is that actually needed on your end? This would only affect people on graphql < 1.12 since the other change is in CI only.

@dipth
Copy link
Contributor Author

dipth commented Feb 16, 2023

You're right. We're doing a number of dependency upgrades ahead of upgrading to Ruby 3.2, so I must have mixed this one together with another one that did have actual code changes :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants