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 csv to gemspec #609

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

andyundso
Copy link

Summary

This addresses the following warning:
warning: csv was loaded from the standard library, but will no longer be part of the default gems since Ruby 3.4.0. Add csv to your Gemfile or gemspec. Also contact author of roo-2.10.1 to add csv into its gemspec.

Closes #604

This addresses the following warning:
warning: csv was loaded from the standard library, but will no longer be part of the default gems since Ruby 3.4.0. Add csv to your Gemfile or gemspec. Also contact author of roo-2.10.1 to add csv into its gemspec.
@andyundso andyundso marked this pull request as ready for review April 2, 2024 06:05
@@ -23,6 +23,7 @@ Gem::Specification.new do |spec|
spec.required_ruby_version = ">= 2.7.0"
end

spec.add_dependency 'csv', '~> 3'
Copy link

@taketo1113 taketo1113 Sep 6, 2024

Choose a reason for hiding this comment

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

@andyundso It seems it needs to add dependency of base64.

The base64 gem warn same message, warning message details is following:

/path/roo/lib/roo.rb:9: warning: base64 was loaded from the standard library, but will no longer be part of the default gems starting from Ruby 3.4.0.
You can add base64 to your Gemfile or gemspec to silence this warning.
Suggested change
spec.add_dependency 'csv', '~> 3'
spec.add_dependency 'base64'
spec.add_dependency 'csv', '~> 3'

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
4 participants