Skip to content

Merge branch 'develop' into chore/vrf-881-vrf-882 #54915

Merge branch 'develop' into chore/vrf-881-vrf-882

Merge branch 'develop' into chore/vrf-881-vrf-882 #54915

Triggered via push February 24, 2024 15:14
Status Success
Total duration 5m 12s
Artifacts

solidity.yml

on: push
Detect changes
9s
Detect changes
Prepublish Test
1m 22s
Prepublish Test
Native Compilation
4m 45s
Native Compilation
Solidity Lint
39s
Solidity Lint
Prettier Formatting
49s
Prettier Formatting
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|21939770381,Native Compilation|21939773471,Prepublish Test|21939773530,Solidity Lint|21939773582,Prettier Formatting|21939773654 , 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#L2894
'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: contracts/test/v0.8/automation/KeeperRegistrar2_0.test.ts#L40
Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free
Solidity Lint
Fallback should be external and payable to accept native currency
Solidity Lint
Constant variable c_feedParamKey is not in all caps, it should be C_FEEDPARAMKEY
Solidity Lint
Explicitly mark visibility of state
Solidity Lint
Constant variable c_timeParamKey is not in all caps, it should be C_TIMEPARAMKEY
Solidity Lint
Explicitly mark visibility of state
Solidity Lint
Constant variable scale is not in all caps, it should be SCALE
Solidity Lint
Explicitly mark visibility of state
Solidity Lint
Internal function getReport is not prefixed with underscore (_)
Solidity Lint
Internal function bytes32ToHexString is not prefixed with underscore (_)
Solidity Lint
Internal function updateFeed is not prefixed with underscore (_)
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/.