-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Fix example for adding local_filter_middleware to middlewares #1253
Conversation
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 @dylanjw!
docs/middleware.rst
Outdated
@@ -383,7 +383,7 @@ retrieved using JSON-RPC endpoints that don't rely on server state. | |||
>>> from web3 import Web3, EthereumTesterProvider | |||
>>> w3 = Web3(EthereumTesterProvider) |
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.
I think EthereumTesterProvider should have parens too?
w3 = Web3(EthereumTesterProvider())
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.
Doh! Thanks.
6c6d172
to
3e3e7dc
Compare
docs/middleware.rst
Outdated
@@ -381,9 +381,9 @@ retrieved using JSON-RPC endpoints that don't rely on server state. | |||
.. code-block:: python |
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.
Can we update these to be doctests so we catch these types of errors in CI. Probably good to revert the changes and confirm the doctest failures and then re-apply the fixes to ensure that doctest is doing what it's supposed to.
@dylanjw do you have time to convert this to a doctest? No worries if not :) |
@kclowes I will try to get to it later this week. |
3e3e7dc
to
6afbd1b
Compare
What was wrong?
Related to Issue #1244
How was it fixed?
Removed the parentheses.
Cute Animal Picture