Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Fix assigned but unused variable warning #326

Merged
merged 1 commit into from

2 participants

@timcraft

Loading webmock with a verbose warning level e.g. ruby -W2 -r webmock -e '' prints out "warning: assigned but unused variable - response". Patch removes the unused variable in question.

@bblimke
Owner

Thanks! In what situation do you load WebMock with verbose warning level?

@bblimke bblimke merged commit f6ad275 into bblimke:master

1 check passed

Details default The Travis CI build passed
@timcraft timcraft deleted the unknown repository branch
@timcraft

@bblimke Thank you, usually every time I run my tests :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Nov 3, 2013
  1. @timcraft
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  lib/webmock/http_lib_adapters/net_http.rb
View
2  lib/webmock/http_lib_adapters/net_http.rb
@@ -97,7 +97,7 @@ def request(request, body = nil, &block)
response = super(request, nil, &nil)
after_request.call(response)
}
- response = if started?
+ if started?
if WebMock::Config.instance.net_http_connect_on_start
super_with_after_request.call
else
Something went wrong with that request. Please try again.