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
Marshal monkey-patch deprecation #621
Conversation
unless ::RSpec::Mocks.configuration.marshal_patched? | ||
RSpec.warn_deprecation(<<-EOS.gsub(/^\s+\|/, '')) | ||
|Using Marshal.dump on stubbed objects relies on a monkey-patch | ||
|that is being made opt-in in RSpec 3. To enable it: |
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.
"To enable it" should perhaps be "To enable it (and silence this warning)"
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.
or 'To silence this warning please explicitly enable it:'
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.
Even better.
LGTM. If we change the API in #620 to be an attribute like I suggested, it'll need to change here as well, of course, but otherwise merge away. |
comments addressed. |
LGTM. If you do decide to add a reader to 3.0, it'd be good to use the same name here (rather than the shortened |
Marshal monkey-patch deprecation
…-deprecation Marshal monkey-patch deprecation --- This commit was imported from rspec/rspec-mocks@678ba90.
For #620