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 helper for running class_exec with keywords when needed #407
Conversation
a82ab9c
to
97d3289
Compare
97d3289
to
8c3a1bb
Compare
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.
🎉
Merged because something odd is going on with the sub builds which means this wasn't picked up on the pinned branch. |
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.
Looks good!
Sorry for the late appearance, hard day at work.
Add helper for running class_exec with keywords when needed
On this subject https://discuss.rubyonrails.org/t/new-2-7-3-0-keyword-argument-pain-point/74980 (cc @JonRowe @pirj )
|
I've added a reply |
…s-class-exec Add helper for running class_exec with keywords when needed --- This commit was imported from rspec/rspec-support@ca5db55.
This is an extraction from a pr on rspec-core but its going to be a repeated pattern to fix keyword arguments across the gem so I'm adding it here. I expect to add more as I find other places that keyword arguments don't work quite right.
The aim will be to keep the string eval to a minimum and this will all be replaced with proper
**kwargs
in RSpec 4, which I want to release in the later half of this year before the next version of Ruby.