Skip to content

Avoid passing named_keys into runtime::call_host_ methods.#2433

Merged
casperlabs-bors-ng[bot] merged 6 commits intocasper-network:devfrom
nilz3ro:GOV-114-b
Jan 12, 2022
Merged

Avoid passing named_keys into runtime::call_host_ methods.#2433
casperlabs-bors-ng[bot] merged 6 commits intocasper-network:devfrom
nilz3ro:GOV-114-b

Conversation

@nilz3ro
Copy link
Contributor

@nilz3ro nilz3ro commented Dec 2, 2021

In this PR I have removed named keys from the signatures of the following functions:

  • Runtime::call_host_mint()
  • Runtime::call_host_auction()
  • Runtime::call_host_handle_payment()

Copy link
Contributor

@darthsiroftardis darthsiroftardis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

Copy link
Collaborator

@mpapierski mpapierski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but you need to fix clippy issues @nilz3ro

@nilz3ro
Copy link
Contributor Author

nilz3ro commented Jan 12, 2022

bors r+

@casperlabs-bors-ng
Copy link
Contributor

Build succeeded:

@casperlabs-bors-ng casperlabs-bors-ng bot merged commit 731212f into casper-network:dev Jan 12, 2022
dwerner pushed a commit to dwerner/casper-node that referenced this pull request Apr 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants