ERC-6900 RI Test CI

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

Sign in to view logs

Triggered via pull request June 19, 2024 19:49

howydev

synchronize #72

howy/add-usdc-token-limit-plugin

Status Success

Total duration 51s

Artifacts –

test.yml

on: pull_request

Run Forge Tests (optimized-test-deep)

Run Forge Tests (default)

Annotations

6 warnings

Run Linters

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, pnpm/action-setup@v2, actions/setup-node@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

Run Linters: src/plugins/ERC20TokenLimitPlugin.sol#L31

Function order is incorrect, struct definition can not go after state variable declaration (line 29)

Run Linters: src/plugins/ERC20TokenLimitPlugin.sol#L62

Possible reentrancy vulnerabilities. Avoid state changes after transfer

Run Linters: src/plugins/NativeTokenLimitPlugin.sol#L85

Function order is incorrect, external function can not go after external pure function (line 79)

Run Forge Tests (default)

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

Run Forge Tests (optimized-test-deep)

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.