Preparing draft PR for review with deployer that uses both a ledger a… #207
ci.yml
on: push
Install dependencies
1m 58s
Benchmark
13m 59s
Typescript lint
16s
Solidity lint
17s
Test contracts
3m 52s
Slither test
1m 47s
Is latest build commited?
3m 34s
Annotations
17 warnings
Install dependencies
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-node@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
Typescript lint
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-node@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
Solidity lint
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-node@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
Solidity lint:
src/contracts/Factory.sol#L3
global import of path @openzeppelin/contracts/access/AccessControl.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
|
Solidity lint:
src/contracts/Factory.sol#L4
global import of path ./Wallet.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
|
Solidity lint:
src/contracts/IWalletProxy.sol#L3
Compiler version ^0.8.3 does not satisfy the 0.8.17 semver requirement
|
Solidity lint:
src/contracts/mocks/CallReceiverMock.sol#L9
Explicitly mark visibility of state
|
Solidity lint:
src/contracts/mocks/CallReceiverMock.sol#L11
Code contains empty blocks
|
Solidity lint:
src/contracts/mocks/ERC165CheckerMock.sol#L6
Explicitly mark visibility of state
|
Solidity lint:
src/contracts/mocks/ERC165CheckerMock.sol#L7
Explicitly mark visibility of state
|
Solidity lint:
src/contracts/mocks/ERC20Mock.sol#L1
Compiler version ^0.8.0 does not satisfy the 0.8.17 semver requirement
|
Solidity lint:
src/contracts/mocks/ERC20Mock.sol#L3
global import of path @openzeppelin/contracts/token/ERC20/ERC20.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
|
Solidity lint:
src/contracts/mocks/HookCallerMock.sol#L4
global import of path ../interfaces/receivers/IERC1155Receiver.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
|
Slither test
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-node@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
Is latest build commited?
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-node@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
Test contracts
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-node@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
Benchmark
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-node@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|