Skip to content

chore: fix eslint warnings #343

chore: fix eslint warnings

chore: fix eslint warnings #343

Triggered via pull request November 12, 2024 10:28
Status Success
Total duration 5m 17s
Artifacts

checks.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

18 warnings
Lint
Unexpected input(s) 'node-version', valid inputs are ['version', 'dest', 'run_install', 'package_json_file', 'standalone']
Lint
Unexpected input(s) 'node-version', valid inputs are ['version', 'dest', 'run_install', 'package_json_file', 'standalone']
Lint: packages/lib/modules/pool/actions/LiquidityActionHelpers.integration.spec.ts#L102
'poolId' is assigned a value but never used
Lint: packages/lib/modules/pool/actions/LiquidityActionHelpers.integration.spec.ts#L217
'poolId' is assigned a value but never used
Lint: packages/lib/modules/tokens/TokenRow/TokenRow.tsx#L149
React Hook useEffect has missing dependencies: 'abbreviated', 'isBpt', 'pool', 'token', and 'usdValueForToken'. Either include them or remove the dependency array. If 'setUsdValue' needs the current value of 'pool', you can also switch to useReducer instead of useState and read 'pool' in the reducer
Lint: packages/lib/modules/tokens/TokenSelectModal/TokenSelectList/TokenSelectList.tsx#L183
Do not define components during render. React will see a new component type on every render and destroy the entire subtree’s DOM nodes and state (https://reactjs.org/docs/reconciliation.html#elements-of-different-types). Instead, move this component definition out of the parent component “TokenSelectList” and pass data as props. If you want to allow component creation in props, set allowAsProps option to true
Lint: packages/lib/modules/vebal/lock/form/VebalLockForm.tsx#L72
React Hook useEffect has a missing dependency: 'onEditAmountToggle'. Either include it or remove the dependency array
Lint: packages/lib/shared/components/marketing/EcosystemChainSelect.tsx#L68
Do not define components during render. React will see a new component type on every render and destroy the entire subtree’s DOM nodes and state (https://reactjs.org/docs/reconciliation.html#elements-of-different-types). Instead, move this component definition out of the parent component “EcosystemChainSelect” and pass data as props. If you want to allow component creation in props, set allowAsProps option to true
Lint: packages/lib/shared/utils/currencies.ts#L39
PRIVATE_CURRENCYAPI_KEY is not listed as a dependency in turbo.json
Lint: packages/lib/test/anvil/anvil-setup.ts#L77
VITEST_POOL_ID is not listed as a dependency in turbo.json
Integration-Test
Unexpected input(s) 'node-version', valid inputs are ['version', 'dest', 'run_install', 'package_json_file', 'standalone']
Integration-Test
Unexpected input(s) 'node-version', valid inputs are ['version', 'dest', 'run_install', 'package_json_file', 'standalone']
Unit-Test
Unexpected input(s) 'node-version', valid inputs are ['version', 'dest', 'run_install', 'package_json_file', 'standalone']
Unit-Test
Unexpected input(s) 'node-version', valid inputs are ['version', 'dest', 'run_install', 'package_json_file', 'standalone']
Build
Unexpected input(s) 'node-version', valid inputs are ['version', 'dest', 'run_install', 'package_json_file', 'standalone']
Build
Unexpected input(s) 'node-version', valid inputs are ['version', 'dest', 'run_install', 'package_json_file', 'standalone']