Skip to content

[5/n permissions] feat: add erc20 token spend limit plugin #383

[5/n permissions] feat: add erc20 token spend limit plugin

[5/n permissions] feat: add erc20 token spend limit plugin #383

Triggered via pull request July 8, 2024 17:44
Status Success
Total duration 47s
Artifacts

test.yml

on: pull_request
Run Linters
25s
Run Linters
Run Forge Tests (optimized-test-deep)
38s
Run Forge Tests (optimized-test-deep)
Run Forge Tests (default)
32s
Run Forge Tests (default)
Fit to window
Zoom out
Zoom in

Annotations

5 warnings
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/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 (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 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/