Skip to content

Fix bug where we were making 2 zkevms for the second fork instead of 1 zkevm and 1 bridge #263

Fix bug where we were making 2 zkevms for the second fork instead of 1 zkevm and 1 bridge

Fix bug where we were making 2 zkevms for the second fork instead of 1 zkevm and 1 bridge #263

Triggered via pull request November 22, 2023 09:20
Status Success
Total duration 38s
Artifacts

test.yml

on: pull_request
Foundry project
29s
Foundry project
Matrix: Code linting
Fit to window
Zoom out
Zoom in

Annotations

4 warnings
Code linting (16.x, ubuntu-latest)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Code linting (16.x, ubuntu-latest): test/ChainIdManager.t.sol#L78
Visibility modifier must be first in list of modifiers
Code linting (16.x, ubuntu-latest): test/ForkingManager.t.sol#L263
Avoid making time-based decisions in your business logic
Code linting (16.x, ubuntu-latest)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/