Skip to content

add reentrancy guard #57388

add reentrancy guard

add reentrancy guard #57388

Triggered via push March 15, 2024 14:46
Status Success
Total duration 5m 27s
Artifacts

solidity.yml

on: push
Detect changes
6s
Detect changes
Fit to window
Zoom out
Zoom in

Annotations

1 error and 21 warnings
Solidity Lint
Got error Error: No job for job name: "Lint" found during self-lookup, invalid job name given? Available jobs names + ids: Detect changes|22710486673,Prettier Formatting|22710495083,Solidity Lint|22710495339,Prepublish Test|22710495603,Native Compilation|22710495882 , failing action
Solidity Lint: contracts/test/v0.8/Cron.test.ts#L43
'libType' is never reassigned. Use 'const' instead
Solidity Lint: contracts/test/v0.8/KeeperRegistrar.test.ts#L37
Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free
Solidity Lint: contracts/test/v0.8/VRFSubscriptionBalanceMonitor.test.ts#L282
'tx' is never reassigned. Use 'const' instead
Solidity Lint: contracts/test/v0.8/automation/AutomationRegistry2_2.test.ts#L1329
'newConfig' is never reassigned. Use 'const' instead
Solidity Lint: contracts/test/v0.8/automation/AutomationRegistry2_2.test.ts#L1361
'newConfig' is never reassigned. Use 'const' instead
Solidity Lint: contracts/test/v0.8/automation/AutomationRegistry2_2.test.ts#L1462
'newConfig' is never reassigned. Use 'const' instead
Solidity Lint: contracts/test/v0.8/automation/AutomationRegistry2_2.test.ts#L2895
'checkUpkeepResult' is never reassigned. Use 'const' instead
Solidity Lint: contracts/test/v0.8/automation/AutomationRegistry2_3.test.ts#L2917
'checkUpkeepResult' is never reassigned. Use 'const' instead
Solidity Lint: contracts/test/v0.8/automation/ERC20BalanceMonitor.test.ts#L276
'tx' is never reassigned. Use 'const' instead
Solidity Lint: contracts/test/v0.8/automation/EthBalanceMonitor.test.ts#L267
'tx' is never reassigned. Use 'const' instead
Solidity Lint
Fallback should be external and payable to accept native currency
Solidity Lint
Immutable variable LINK is not prefixed with 'i_'
Solidity Lint
Avoid to use low level calls
Solidity Lint
Contract has 27 states declarations but allowed no more than 15
Solidity Lint
Avoid to use tx.origin
Solidity Lint
Avoid to use low level calls
Solidity Lint
Avoid to use low level calls
Solidity Lint
global import of path ../UpkeepFormat.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
Solidity Lint
global import of path ../UpkeepFormat.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
Solidity Lint
Fallback should be external and payable to accept native currency
Native Compilation
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.