Merge branch 'develop' into willc/delete-flags

3a126a19 · mergify[bot] · GitHub · 4006ef3a · 03323455 · 3a126a19
Commit 3a126a19 authored Oct 25, 2022 by mergify[bot] Committed by GitHub Oct 25, 2022
Expand all Hide whitespace changes
Inline Side-by-side

Showing with 67 additions and 32 deletions

README.md README.md +65 -30

CrossDomainMessenger.t.sol ...ntracts-bedrock/contracts/test/CrossDomainMessenger.t.sol +2 -2

No files found.
--- a/README.md
+++ b/README.md
--- a/packages/contracts-bedrock/contracts/test/CrossDomainMessenger.t.sol
+++ b/packages/contracts-bedrock/contracts/test/CrossDomainMessenger.t.sol
@@ -8,12 +8,12 @@ import { Messenger_Initializer, Reverter, CallerCaller } from "./CommonTest.t.so
 contract CrossDomainMessenger_Test is Messenger_Initializer {
    // Ensure that baseGas passes for the max value of _minGasLimit,
    // this is about 4 Billion.
-    function test_baseGas() external {
+    function test_baseGas() external view {
        L1Messenger.baseGas(hex"ff", type(uint32).max);
    }

    // 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);
    }
 }