Skip to content

Conversation

@grasphoper
Copy link
Contributor

When trying to run the version in master, I get Error HH310: Invalid param --spokePool. Command line params must be lowercase.

This is why I changed params to lowercase.

FYI @bmzig

Signed-off-by: Ihor Farion <ihor@umaproject.org>
@bmzig
Copy link
Contributor

bmzig commented Mar 21, 2025

This isn't an issue per se. You are supposed to run hardhat tasks with camel case like yarn evm-relay-message-withdrawal --spoke-pool [spoke_pool_address] --adapter [adapter_address] --l2-token [l2_token_address] --amount-to-return [amount_to_return]. With that being said, the comment on how to run it was indeed incorrect.

Signed-off-by: Ihor Farion <ihor@umaproject.org>
Signed-off-by: Ihor Farion <ihor@umaproject.org>
Signed-off-by: Ihor Farion <ihor@umaproject.org>
@grasphoper
Copy link
Contributor Author

This isn't an issue per se. You are supposed to run hardhat tasks with camel case like yarn evm-relay-message-withdrawal --spoke-pool [spoke_pool_address] --adapter [adapter_address] --l2-token [l2_token_address] --amount-to-return [amount_to_return]. With that being said, the comment on how to run it was indeed incorrect.

Right, that's weird. Reverted back the name changes. Changed the comment to reflect the correct command

@grasphoper grasphoper changed the title Fix hardhat task: evm-relay-message-withdrawal fix(tasks): fix usage comment + typo in evm-relay-message-withdrawal Mar 21, 2025
@grasphoper grasphoper merged commit 437eee0 into master Mar 21, 2025
9 checks passed
@grasphoper grasphoper deleted the if/fix-evm-relay-message-withdrawal-task branch March 21, 2025 00:50
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.

3 participants