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

Ruby 3.5 compatibility #126

Merged
merged 3 commits into from
Sep 24, 2024
Merged

Conversation

padde
Copy link
Contributor

@padde padde commented Sep 11, 2024

Fixes #121

The ostruct gem was historically included in the default gems included with Ruby. Starting with Ruby 3.3.5 though, requiring ostruct without explicitly adding it as dependency generates the following warning:

warning: ostruct was loaded from the standard library, but will no longer be part of the default gems starting from Ruby 3.5.0.
You can add ostruct to your Gemfile or gemspec to silence this warning.

As of Ruby 3.5, the ostruct gem will then no longer be shipped with Ruby so we need to explicitly add it to the dependencies.

Blockers

TODO

  • Add Ruby 3.3 to CI matrix

@padde padde changed the title Ruby 3.5 compatibility Ruby 3.3 compatibility Sep 11, 2024
@padde padde changed the title Ruby 3.3 compatibility Ruby 3.5 compatibility Sep 11, 2024
@padde padde marked this pull request as ready for review September 11, 2024 22:48
@padde padde force-pushed the ruby-3-5-compatibility branch from 6102b22 to 3a0b6d6 Compare September 15, 2024 20:42
@grape-bot
Copy link

1 Message
📖 We really appreciate pull requests that demonstrate issues, even without a fix. That said, the next step is to try and fix the failing tests!

Generated by 🚫 Danger

@padde padde force-pushed the ruby-3-5-compatibility branch from 3a0b6d6 to b54982f Compare September 21, 2024 11:51
@dblock
Copy link
Member

dblock commented Sep 21, 2024

Want to rebase this one?

This gem was historically included in the default gemset. Starting with
Ruby 3.3.5 though, requiring `ostruct` without explicitly adding it as
dependency generates the following warning:

    warning: ostruct was loaded from the standard library, but will no longer be part of the default gems starting from Ruby 3.5.0.
    You can add ostruct to your Gemfile or gemspec to silence this warning.
@padde padde force-pushed the ruby-3-5-compatibility branch from b54982f to abc1512 Compare September 21, 2024 19:31
@padde
Copy link
Contributor Author

padde commented Sep 21, 2024

@dblock done, please review.

@dblock
Copy link
Member

dblock commented Sep 24, 2024

Good work @padde! Interested in helping out with maintaining this library, maybe make the next release? If yes drop me an email to dblock at dblock dot org with your rubygems username.

@dblock dblock merged commit e0d9a56 into ruby-grape:master Sep 24, 2024
21 checks passed
@padde padde deleted the ruby-3-5-compatibility branch September 24, 2024 05:01
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 regarding ostruct on Ruby >= 3.3.5
3 participants