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

gemspec: Require wasabi version that supports Ruby 3 #997

Merged

Conversation

sarahsehr
Copy link
Contributor

@sarahsehr sarahsehr commented Dec 15, 2023

What kind of change is this?

Bugfix

Did you add tests for your changes?

No. There does not appear to be a way to test against different combinations of gems.

Summary of changes

Wasabi < 3.6 uses URI.unescape which does not exist in Ruby 3. Wasabi does not test against Ruby 3 until v3.7, which is why the minimum is set to 3.7.

#996

Other information

Recommend to patch 2.13 and 2.14 because they both support Ruby 3 according to the CI config.

@sarahsehr sarahsehr marked this pull request as ready for review December 15, 2023 18:26
@olleolleolle
Copy link
Contributor

@sarahsehr Thank you for fixing!

@olleolleolle olleolleolle merged commit 8335f46 into savonrb:master Dec 21, 2023
7 checks passed
@sarahsehr sarahsehr deleted the fix-wasabi-version-for-ruby-3 branch December 21, 2023 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants