Further tidy up of the code #208
ci.yml
on: push
Install dependencies
15s
Benchmark
14m 51s
Typescript lint
22s
Solidity lint
15s
Test contracts
3m 41s
Slither test
1m 44s
Is latest build commited?
3m 52s
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/
|
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)
|
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/
|
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/
|
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/
|
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/
|
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/
|