[DRAFT] [6/n permissions] feat: add cold storage plugin #384
test.yml
on: pull_request
Run Linters
29s
Run Forge Tests (optimized-test-deep)
41s
Run Forge Tests (default)
24s
Annotations
6 warnings
Run Forge Tests (default)
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 Linters
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, pnpm/action-setup@v2, 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 Linters:
src/plugins/ColdStoragePlugin.sol#L112
Function order is incorrect, external view function can not go after internal view function (line 77)
|
Run Linters:
src/plugins/ERC20TokenLimitPlugin.sol#L50
Function order is incorrect, external function can not go after external view function (line 42)
|
Run Linters:
src/plugins/ERC20TokenLimitPlugin.sol#L67
Possible reentrancy vulnerabilities. Avoid state changes after transfer
|
Run Forge Tests (optimized-test-deep)
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/
|