Skip to content

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