Skip to content

Add 'ostruct' to example app Gemfile #2796

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

Closed
wants to merge 1 commit into from
Closed

Add 'ostruct' to example app Gemfile #2796

wants to merge 1 commit into from

Conversation

davidrunger
Copy link
Contributor

Currently, specs are failing for the "Ruby: 3.3, Rails: ~> 7.1.0" and "Ruby: 3.3, Rails: ~> 7.2.0" matrix builds. This is caused by Ruby 3.3.5 (released yesterday, 2024-09-03) now warning if ostruct is loaded from the standard library, rather than from a gem.

This change fixes the failing specs by adding the ostruct gem to the Gemfile of the example app, which causes the problematic warning not to be emitted by Ruby.

Currently, specs are failing for the "Ruby: 3.3, Rails: ~> 7.1.0" and
"Ruby: 3.3, Rails: ~> 7.2.0" matrix builds. This is caused by Ruby 3.3.5
(released yesterday, 2024-09-03) now warning if `ostruct` is loaded from
the standard library, rather than from a gem.

This change fixes the failing specs by adding the `ostruct` gem to the
`Gemfile` of the example app, which causes the problematic warning not
to be emitted by Ruby.
@davidrunger
Copy link
Contributor Author

I think that #2794 aims to solve the same problem that this PR does (i.e. currently failing specs for some matrix builds on main), so, if that PR is merged, then that will render this PR unnecessary, and it should be closed.

@JonRowe JonRowe closed this Sep 4, 2024
@JonRowe
Copy link
Member

JonRowe commented Sep 4, 2024

👋 Thanks but its actually not just ostruct and is actually Rails issue upstream

@davidrunger davidrunger deleted the add-ostruct-to-example-app-gemfile branch September 4, 2024 20:47
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