-
-
Notifications
You must be signed in to change notification settings - Fork 241
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
refactor: Prefer Model.reflections over private Model#_reflections. #2927
Conversation
- `_reflections` returns different key type per Rails version
@Paul-Bob please review this and it works, let's ship it for 2.0! We should check out if we can port it to 3.0 as well. |
I can update also to |
OK. let's retry that one then. Maybe it works on your end! |
This PR has been marked as stale because there was no activity for the past 15 days. |
Made this PR for version 3 that replace |
@Paul-Bob updated, I tried to keep changes minimal. Specs seems happy. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @simi!
Let's ship this.
🚧 This targets 2.x branch.
_reflections
returns different key type per Rails versionℹ️ Model.reflections seems supported for 6.0+ (I have not checked earlier versions since those seems unsupported).
💡 This fixes issues at RubyGems.org with Rails 7.2 beta compatibility.