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.

Sign up for GitHub

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

Jump to bottom

Address "literal string will be frozen in the future" warnings #15

Open

StalemateInc wants to merge 1 commit into 29decibel:master from StalemateInc:fix-frozen-string-literal-warning

StalemateInc commented Dec 28, 2024

Should resolve #14.

dup the initial result to avoid freezing and failing down the codepaths that modify it.
Fix opening the URL by swapping open with URI.open - Kernel monkey-patch has been removed from open-uri long time ago. I did not test it locally with Rubies that predate 2.6.6.


          Make the result not frozen since it's going to be modified; fix the…

e3d119b

… url opening with open-uri

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet