Skip to content

Commit

Permalink
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
refactor: needless kwarg pop
Browse files Browse the repository at this point in the history
antazoey committed Jan 31, 2025
1 parent 301be36 commit 905928c
Showing 2 changed files with 2 additions and 4 deletions.
3 changes: 1 addition & 2 deletions src/ape/api/accounts.py
Original file line number Diff line number Diff line change
@@ -258,8 +258,7 @@ def transfer(
if txn.value < 0:
raise AccountsError("Value cannot be negative.")

sign = kwargs.pop("sign", True)
return self.call(txn, private=private, sign=sign, **kwargs)
return self.call(txn, private=private, **kwargs)

def deploy(
self, contract: "ContractContainer", *args, publish: bool = False, **kwargs
3 changes: 1 addition & 2 deletions src/ape/contracts/base.py
Original file line number Diff line number Diff line change
@@ -90,8 +90,7 @@ def __call__(self, private: bool = False, *args, **kwargs) -> "ReceiptAPI":

if "sender" in kwargs and hasattr(kwargs["sender"], "call"):
sender = kwargs["sender"]
sign = kwargs.pop("sign", True)
return sender.call(txn, sign=sign, **kwargs)
return sender.call(txn, **kwargs)
elif "sender" not in kwargs and self.account_manager.default_sender is not None:
return self.account_manager.default_sender.call(txn, **kwargs)

0 comments on commit 905928c

Please sign in to comment.