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

support ruby 3.1 and keep old versions #15

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kritik
Copy link

@kritik kritik commented Mar 14, 2023

No description provided.

@kritik
Copy link
Author

kritik commented Mar 14, 2023

I know about #10. Difference is that my version keeps support for old versions as well

@haseebeqx
Copy link

haseebeqx commented Apr 8, 2023

@dball Please merge this. This PR will make this gem support newer versions of ruby.

@ashgkwd
Copy link

ashgkwd commented May 16, 2024

I need this PR merged. Can someone please merge it?

I get the following error without this fix.

NameError: uninitialized class variable @@schemes in URI
/Users/ash/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/data_uri-0.1.0/lib/data_uri/uri.rb:64:in `<module:URI>'

will-clarke-blacklane pushed a commit to will-clarke-blacklane/data_uri that referenced this pull request Nov 8, 2024
@PikachuEXE
Copy link

Planning to release a new gem with similar name just to let people use it with newer uri versions (required by rails 8)
data-uri-ruby#1 to be merged within this month & start releasing first version

@PikachuEXE
Copy link

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.

5 participants