Skip to content

Commit

Permalink
Merge #17030: test: Fix Python Docstring to include all Args.
Browse files Browse the repository at this point in the history
8acd589 Fix Python Docstring to include all Args. (John Bampton)

Pull request description:

  Found a Python function that had incorrect and missing arguments in its Docstring.

ACKs for top commit:
  laanwj:
    ACK 8acd589

Tree-SHA512: 936f275f29a700d630bb479b5283e47b66f2df76d8b8c053f594e6aedf783cc98a29c924c3a46613f112dfc884acb50f21a0b18f96d939e887b12b921ef2e10f
  • Loading branch information
laanwj committed Oct 8, 2019
2 parents febc31c + 8acd589 commit 866fd28
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion test/functional/test_framework/util.py
Original file line number Diff line number Diff line change
Expand Up @@ -244,14 +244,15 @@ class PortSeed:
# Must be initialized with a unique integer for each process
n = None

def get_rpc_proxy(url, node_number, timeout=None, coveragedir=None):
def get_rpc_proxy(url, node_number, *, timeout=None, coveragedir=None):
"""
Args:
url (str): URL of the RPC server to call
node_number (int): the node number (or id) that this calls to
Kwargs:
timeout (int): HTTP timeout in seconds
coveragedir (str): Directory
Returns:
AuthServiceProxy. convenience object for making RPC calls.
Expand Down

0 comments on commit 866fd28

Please sign in to comment.