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 base64 to runtime dependencies for Ruby 3.4 compatibility #476

Merged
merged 2 commits into from
Jul 24, 2024

Conversation

tisba
Copy link
Contributor

@tisba tisba commented Jul 24, 2024

See https://github.com/ruby/ruby/blob/master/doc/NEWS/NEWS-3.3.0.md#stdlib-updates

With Ruby 3.4 on forward, base64 will no longer be available as part of part of the default gems. It needs to be an explicit dependency going forward.

With Ruby 3.4, base64 will no longer be available as part of part of the default gems. It needs to be an explicit dependency going forward.
@tisba
Copy link
Contributor Author

tisba commented Jul 24, 2024

Dear maintainers, let me know if you want me to fix the linting errors too. I kept the change in this PR to a minimum for now.

@route
Copy link
Member

route commented Jul 24, 2024

Let's fix linter and I'll merge!

@tisba
Copy link
Contributor Author

tisba commented Jul 24, 2024

Linters are happy. The now failing Ferrum::ProcessTimeoutError for Ruby 2.7 looks like an intermittent error.

@route
Copy link
Member

route commented Jul 24, 2024

Thanks!

@route route merged commit a5275bf into rubycdp:main Jul 24, 2024
7 checks passed
@tisba tisba deleted the patch-1 branch July 24, 2024 16:37
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.

2 participants