CM-778 - Add multicaller contract with tests #769
alejoloaizatest.yml
on: pull_request
Run Forge Tests
49s
Run Hardhat Tests
2m 44s
Run eslint
4m 9s
Run solhint
2m 7s
Run slither
6m 17s
Check README.md is contained in each contracts directory
3s
Publish to NPM (dry run)
2m 20s
Annotations
21 errors and 17 warnings
|
Run solhint:
contracts/mocks/MockFunctions.sol#L5
Replace ⏎ with ··
|
|
|
|
|
|
|
|
Run solhint:
contracts/mocks/MockFunctions.sol#L11
Replace ···· with ········
|
|
Run solhint:
contracts/mocks/MockFunctions.sol#L11
Use Custom Errors instead of revert statements
|
|
|
|
|
|
|
|
Run solhint:
contracts/mocks/MockFunctions.sol#L16
Replace }⏎ with ··}
|
|
Publish to NPM (dry run):
test/multicall/GuardedMulticaller.test.ts#L66
No overload matches this call.
|
|
Publish to NPM (dry run):
test/multicall/GuardedMulticaller.test.ts#L77
No overload matches this call.
|
|
Publish to NPM (dry run):
test/multicall/GuardedMulticaller.test.ts#L87
No overload matches this call.
|
|
Publish to NPM (dry run):
test/multicall/GuardedMulticaller.test.ts#L98
No overload matches this call.
|
|
Publish to NPM (dry run):
test/multicall/GuardedMulticaller.test.ts#L108
No overload matches this call.
|
|
Publish to NPM (dry run):
test/multicall/GuardedMulticaller.test.ts#L120
No overload matches this call.
|
|
Publish to NPM (dry run):
test/multicall/GuardedMulticaller.test.ts#L129
No overload matches this call.
|
|
Publish to NPM (dry run):
test/multicall/GuardedMulticaller.test.ts#L138
No overload matches this call.
|
|
Publish to NPM (dry run):
test/multicall/GuardedMulticaller.test.ts#L147
No overload matches this call.
|
|
Publish to NPM (dry run):
test/multicall/GuardedMulticaller.test.ts#L156
No overload matches this call.
|
|
Run slither
Process completed with exit code 255.
|
|
Check README.md is contained in each contracts directory
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Run Forge Tests
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Run solhint
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, 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/
|
|
Run solhint:
contracts/access/MintingAccessControl.sol#L6
Variable "AccessControl" is unused
|
|
Run solhint:
contracts/access/MintingAccessControl.sol#L6
Variable "IAccessControl" is unused
|
|
Run solhint:
contracts/mocks/MockFunctions.sol#L6
Code contains empty blocks
|
|
Run solhint:
contracts/mocks/MockFunctions.sol#L11
Provide an error message for revert
|
|
Run solhint:
contracts/mocks/MockFunctions.sol#L14
Code contains empty blocks
|
|
Run solhint:
contracts/multicall/GuardedMulticaller.sol#L6
global import of path @openzeppelin/contracts/utils/cryptography/SignatureChecker.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
|
|
Run solhint:
contracts/multicall/GuardedMulticaller.sol#L9
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)
|
|
Run solhint:
contracts/multicall/GuardedMulticaller.sol#L12
global import of path @openzeppelin/contracts/security/ReentrancyGuard.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
|
|
Run solhint:
contracts/multicall/GuardedMulticaller.sol#L15
global import of path @openzeppelin/contracts/utils/cryptography/EIP712.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
|
|
Run solhint:
contracts/multicall/GuardedMulticaller.sol#L28
Main key parameter in mapping permittedFunctionSelectors is not named
|
|
Publish to NPM (dry run)
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, 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/
|
|
Run Hardhat Tests
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, 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/
|
|
Run eslint
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, 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/
|
|
Run slither
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|