Commit 3616c249 authored by mergify[bot]'s avatar mergify[bot] Committed by GitHub

Merge branch 'develop' into jg/max_rlp_bytes_per_channel

parents 90b6080c 248908b4
---
'@eth-optimism/indexer': patch
---
Delete unused flags
This diff is collapsed.
...@@ -22,12 +22,6 @@ var ( ...@@ -22,12 +22,6 @@ var (
Required: true, Required: true,
EnvVar: prefixEnvVar("BUILD_ENV"), EnvVar: prefixEnvVar("BUILD_ENV"),
} }
EthNetworkNameFlag = cli.StringFlag{
Name: "eth-network-name",
Usage: "Ethereum network name",
Required: true,
EnvVar: prefixEnvVar("ETH_NETWORK_NAME"),
}
ChainIDFlag = cli.StringFlag{ ChainIDFlag = cli.StringFlag{
Name: "chain-id", Name: "chain-id",
Usage: "Ethereum chain ID", Usage: "Ethereum chain ID",
...@@ -188,7 +182,6 @@ var ( ...@@ -188,7 +182,6 @@ var (
var requiredFlags = []cli.Flag{ var requiredFlags = []cli.Flag{
BuildEnvFlag, BuildEnvFlag,
EthNetworkNameFlag,
ChainIDFlag, ChainIDFlag,
L1EthRPCFlag, L1EthRPCFlag,
L2EthRPCFlag, L2EthRPCFlag,
......
...@@ -8,12 +8,12 @@ import { Messenger_Initializer, Reverter, CallerCaller } from "./CommonTest.t.so ...@@ -8,12 +8,12 @@ import { Messenger_Initializer, Reverter, CallerCaller } from "./CommonTest.t.so
contract CrossDomainMessenger_Test is Messenger_Initializer { contract CrossDomainMessenger_Test is Messenger_Initializer {
// Ensure that baseGas passes for the max value of _minGasLimit, // Ensure that baseGas passes for the max value of _minGasLimit,
// this is about 4 Billion. // this is about 4 Billion.
function test_baseGas() external { function test_baseGas() external view {
L1Messenger.baseGas(hex"ff", type(uint32).max); L1Messenger.baseGas(hex"ff", type(uint32).max);
} }
// Fuzz for other values which might cause a revert in baseGas. // Fuzz for other values which might cause a revert in baseGas.
function testFuzz_baseGas(uint32 _minGasLimit) external { function testFuzz_baseGas(uint32 _minGasLimit) external view {
L1Messenger.baseGas(hex"ff", _minGasLimit); L1Messenger.baseGas(hex"ff", _minGasLimit);
} }
} }
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment