Improved code a bit more #212
ci.yml
on: push
Install dependencies
16s
Benchmark
14m 17s
Typescript lint
17s
Solidity lint
16s
Test contracts
3m 52s
Slither test
1m 28s
Is latest build commited?
3m 53s
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/
|