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

Explicitly require Base64 and CSV #616

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

Conversation

HashNotAdam
Copy link

Summary

Ruby has been trying to reduce the surface area of the standard library. While Base64 and CSV are default gems, starting with Ruby 3.4, they are no longer included in the standard library and must be explicitly included as dependencies

Other information

Resolves #604

@Faq
Copy link

Faq commented Jan 25, 2025

@simonoff or @chopraanmol1 can you check this one? ref. #609

@simonoff
Copy link
Member

@HashNotAdam can you also add a ruby 3.4 into pipeline? Also, i think would be great to add such gems only after 3.3 ruby.

HashNotAdam and others added 2 commits January 26, 2025 16:38
Ruby has been trying to reduce the surface area of the standard library.
While Base64 and CSV are default gems, starting with Ruby 3.4,they are
no longer included in the standard library and must be explicitly
included as dependencies
- ruby-head
- jruby-9.3.3.0
- jruby-9.4.10.0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No such version

@simonoff
Copy link
Member

@HashNotAdam can you fix jruby version and add a changelog please?

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.

Warnings are displayed when roo is used with Ruby 3.3.0
3 participants