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

Declare base64 gem dependency, ready for Ruby 3.4. #308

Merged
merged 1 commit into from
May 15, 2024
Merged

Declare base64 gem dependency, ready for Ruby 3.4. #308

merged 1 commit into from
May 15, 2024

Conversation

sengi
Copy link
Contributor

@sengi sengi commented May 15, 2024

The base64 gem is no longer a default gem in Ruby 3.4.

Add base64 to the gemspec, for forward compatibility with Ruby 3.4 and to resolve the base64 was loaded from the standard library, but will no longer be part of the default gems since Ruby 3.4.0 warning that Ruby 3.3 emits on loading client_ruby.

The base64 gem is no longer a default gem in Ruby 3.4.

Add base64 to the gemspec, for forward compatibility with Ruby 3.4 and
to resolve the `base64 was loaded from the standard library, but will no
longer be part of the default gems since Ruby 3.4.0` warning that Ruby
3.3 emits on loading client_ruby.

https://docs.ruby-lang.org/en/master/NEWS_md.html#label-Stdlib+updates

Signed-off-by: Chris Banks <chris@banks.cx>
Copy link
Member

@Sinjo Sinjo left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this!

I'm gonna ignore that JRuby 9.1 build being sad and force the merge through. I don't know exactly what's going on there, but that version currently gets special treatment and I think it's time to say goodbye to it.

@Sinjo Sinjo merged commit c5f87c5 into prometheus:main May 15, 2024
10 of 11 checks passed
@sengi
Copy link
Contributor Author

sengi commented May 15, 2024

Thanks @Sinjo! Was just having a look into that test failure myself. I think it may be a similar issue, this time with tins gem (:test dependency in Gemfile) using bigdecimal but not declaring it yet. Not sure why JRuby 9.1 in particular doesn't like it though.

@sengi
Copy link
Contributor Author

sengi commented May 15, 2024

Sorry, scratch that last part, tins does declare its dependency on bigdecimal.

@xeruzo
Copy link

xeruzo commented Jul 4, 2024

it adds a new question for those use ruby < 2.4...
#315

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.

3 participants