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

Test on Ruby 3.3.0 #1014

Closed
wants to merge 1 commit into from
Closed

Test on Ruby 3.3.0 #1014

wants to merge 1 commit into from

Conversation

benlangfeld
Copy link
Contributor

In the hopes that it exposes #1013

In the hopes that it exposes pivotal#1013
@pivotal-cla
Copy link

@benlangfeld Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@cf-gitbot
Copy link
Collaborator

We have created an issue in Pivotal Tracker to manage this. Unfortunately, the Pivotal Tracker project is private so you may be unable to view the contents of the story.

The labels on this github issue will be updated when the story is started.

@pivotal-cla
Copy link

@benlangfeld Thank you for signing the Contributor License Agreement!

benlangfeld added a commit to benlangfeld/LicenseFinder that referenced this pull request Jan 12, 2024
On Ruby 3.3 the following warning is emitted:

```
/usr/local/bundle/ruby/3.3.0/gems/license_finder-7.1.0/lib/license_finder/package_managers/sbt.rb:3: 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 license_finder-7.1.0 to add csv into its gemspec.
```

As of Ruby 3.4 license_finder would cease to work without this explicit dependency.

See also pivotal#1014
@xtreme-shane-lattanzio
Copy link
Contributor

@benlangfeld Ill add 3.3.x support to the pipeline. There are a few things missing from this so Ill make sure pipelines are added and they pass

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

Successfully merging this pull request may close these issues.

4 participants