Skip to content

add notes

add notes #52

Triggered via push July 9, 2024 05:27
Status Success
Total duration 2m 8s
Artifacts

test.yml

on: push
Matrix: Run Unit and E2E Tests
Matrix: Run Linters
Fit to window
Zoom out
Zoom in

Annotations

12 warnings
Run Linters (18.x)
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 Linters (18.x): src/leverage/interfaces/IParaswapSellAdapter.sol#L4
imported name IPoolAddressesProvider is not used
Run Linters (18.x): src/leverage/interfaces/IParaswapSellAdapter.sol#L5
imported name IPool is not used
Run Linters (18.x): src/leverage/ParaswapSellAdapter.sol#L4
global import of path forge-std/Test.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
Run Linters (18.x): src/leverage/ParaswapSellAdapter.sol#L4
Import 'forge-std/Test.sol' in contract ParaswapSellAdapter should be declared as import {contract_to_import} from 'forge-std/Test.sol';
Run Linters (18.x): src/leverage/ParaswapSellAdapter.sol#L14
imported name ISAFEEngine is not used
Run Linters (18.x): src/leverage/ParaswapSellAdapter.sol#L133
Return value 'bool' in function 'executeOperation' must be named
Run Linters (18.x): src/leverage/ParaswapSellAdapter.sol#L134
'asset' should start with _
Run Linters (18.x): src/leverage/ParaswapSellAdapter.sol#L135
'amount' should start with _
Run Linters (18.x): src/leverage/ParaswapSellAdapter.sol#L136
'premium' should start with _
Run Linters (18.x): src/leverage/ParaswapSellAdapter.sol#L137
'initiator' should start with _
Run Unit and E2E Tests (20.x)
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/