Closed jribbink closed 4 months ago
Bit confused, in my reported issue it seemed like the given parameter wasn't being honored. Is something else going on that skipping validation will fix? If I also try to specify evm-network-id
emulator is not a valid input
Bit confused, in my reported issue it seemed like the given parameter wasn't being honored. Is something else going on that skipping validation will fix? If I also try to specify
evm-network-id
emulator is not a valid input
The issue is that cfg.FlowNetworkID
is a flow.ChainID
(from flow-go
) where chains are formatted like: flow-testnet
, flow-mainnet
, etc. However, in that check, these are being compared against strings without the prefix, e.g. emulator
& previewnet
.
The reason it is removed altogether instead of being changed is because this check is redundant and already handled here:
Sorry, should have made this more clear in PR description.
Bit confused, in my reported issue it seemed like the given parameter wasn't being honored. Is something else going on that skipping validation will fix? If I also try to specify
evm-network-id
emulator is not a valid inputThe issue is that
cfg.FlowNetworkID
is aflow.ChainID
(fromflow-go
) where chains are formatted like:flow-testnet
,flow-mainnet
, etc. However, in that check, these are being compared against strings without the prefix, e.g.emulator
&previewnet
.The reason it is removed altogether instead of being changed is because this check is redundant and already handled here:
Sorry, should have made this more clear in PR description.
Thanks for the explanation 🙏
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 42.69%. Comparing base (
9b1830d
) to head (fabb637
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Closes #1620
Description
There were two checks for networks, this removes the problematic one (was comparing
flow-emulator
toemulator
so always failing)For contributor use:
master
branchFiles changed
in the Github PR explorer