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
Update to a more extensible deprecation method. #1043
Update to a more extensible deprecation method. #1043
Conversation
@@ -59,4 +59,4 @@ def certificate_store | |||
end | |||
end | |||
|
|||
EventMachine::HttpStubConnection.send(:include, EmHttpSslPatch) | |||
EventMachine::HttpStubConnection.include(EmHttpSslPatch) |
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.
Rubocop made me do it 🚓 🤖
end | ||
|
||
it 'does not raise an error for inherited error-namespaced classes but prints an error message' do | ||
error_message, = with_warn_squelching { class E < Faraday::Error::ClientError; end } |
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.
this looks like a typo, but it's not. Rubocop likes
a, = [:foo, :bar]
better than
a, _ = [:foo, :bar]
🤷♂
The previous implementation would not warn about deprecation upon being inherited, and warned in a stdout. This implementation warns in stderr similar to Gem::Deprecate, but gives a semver instead of a date like the default Gem::Deprecate does.
fe63747
to
cd7888f
Compare
FYI: This closed because I killed the |
The previous implementation would not warn about deprecation upon being inherited, and warned in stdout. Upon initialization or Inheritance, this implementation warns in stderr similar to Gem::Deprecate, but gives a semver instead of a date like the default Gem::Deprecate does.