ERC-6900 RI Test CI

[4/n permissions] feat: add native token spend limit plugin #224

Sign in to view logs

Triggered via pull request June 13, 2024 16:40

howydev

synchronize #71

howy/add-native-token-limit-plugin

Status Success

Total duration 52s

Artifacts –

test.yml

on: pull_request

Run Forge Tests (optimized-test-deep)

Run Forge Tests (default)

Annotations

5 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/NativeTokenLimitPlugin.sol#L78

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

Run Linters: test/account/AccountPermissionHooks.t.sol#L238

When fallback is not payable you will not be able to receive ether

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/.