Merge branch 'develop' into keystone/add_ocr3_contract_wrapper #57086
HenryNguyen5solidity.yml
on: push
Detect changes
6s
Prepublish Test
1m 28s
Native Compilation
4m 4s
Solidity Lint
48s
Prettier Formatting
47s
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|22628428883,Solidity Lint|22628437270,Native Compilation|22628437518,Prettier Formatting|22628437690,Prepublish Test|22628437894
, 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#L2905
'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/.
|