Skip to content
System contracts CI

Eigenda verifier #114

Sign in to view logs

Eigenda verifier

Eigenda verifier #114

Triggered via pull request October 10, 2024 21:01
@gianbelinchegianbelinche
synchronize #38
eigenda-verifier
Status Failure
Total duration 4m 33s
Artifacts –

system-contracts-ci.yaml

on: pull_request
build
3m 41s
build
lint
45s
lint
check-hashes
31s
check-hashes
test-bootloader
0s
test-bootloader
test-contracts
0s
test-contracts
Fit to window
Zoom out
Zoom in

Annotations

3 errors and 13 warnings
lint: l1-contracts/contracts/eigenda/EigenDAVerifier.sol#L6
global import of path @openzeppelin/contracts/access/Ownable.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
lint: l1-contracts/contracts/eigenda/EigenDAVerifier.sol#L128
Named parameters missing. MIN unnamed argumenst is 4
lint
Process completed with exit code 1.
lint
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
lint: l1-contracts/contracts/state-transition/chain-deps/facets/Executor.sol#L30
Immutable variables name are set to be in capitalized SNAKE_CASE
lint: l1-contracts/contracts/state-transition/chain-deps/facets/Mailbox.sol#L283
Avoid to use tx.origin
lint: l1-contracts/contracts/vendor/AddressAliasHelper.sol#L55
Avoid to use tx.origin
lint: l1-contracts/test/foundry/unit/concrete/AddressAliasHelper/applyL1ToL2Alias.t.sol#L6
Contract name must be in CamelCase
lint: l1-contracts/test/foundry/unit/concrete/AddressAliasHelper/undoL1ToL2Alias.t.sol#L6
Contract name must be in CamelCase
lint: l1-contracts/test/foundry/unit/concrete/Executor/ExecutorProof.t.sol#L2
Found more than One contract per file. 2 contracts found!
lint: l1-contracts/test/foundry/unit/concrete/state-transition/chain-deps/DiamondProxy/DiamondProxy.t.sol#L2
Found more than One contract per file. 2 contracts found!
lint: l1-contracts/test/foundry/unit/concrete/state-transition/chain-deps/facets/Admin/ExecuteUpgrade.t.sol#L3
Found more than One contract per file. 2 contracts found!
lint: l1-contracts/test/foundry/unit/concrete/state-transition/chain-deps/facets/Base/_Base_Shared.t.sol#L2
Found more than One contract per file. 2 contracts found!
lint: l1-contracts/test/foundry/unit/concrete/state-transition/chain-deps/facets/Getters/_Getters_Shared.t.sol#L2
Found more than One contract per file. 2 contracts found!
build
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-node@v3, actions/cache/save@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
check-hashes
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-node@v3, actions/cache/restore@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/