feat(29629): Construct exhaustive state object for metamask slice with default values #58758
Annotations
4 errors and 10 warnings
|
Lint:
shared/types/metamask.ts#L122
Cannot find module '../../app/scripts/lib/accounts/BalancesController' or its corresponding type declarations.
|
|
Lint:
test/data/mock-state.ts#L220
Type 'import("/home/runner/work/metamask-extension/metamask-extension/node_modules/@metamask/utils/dist/versions").SemVerVersion' is not assignable to type 'import("/home/runner/work/metamask-extension/metamask-extension/node_modules/@metamask/base-controller/node_modules/@metamask/utils/dist/versions").SemVerVersion'.
|
|
Lint:
test/data/mock-state.ts#L228
Type 'import("/home/runner/work/metamask-extension/metamask-extension/node_modules/@metamask/utils/dist/versions").SemVerVersion' is not assignable to type 'import("/home/runner/work/metamask-extension/metamask-extension/node_modules/@metamask/base-controller/node_modules/@metamask/utils/dist/versions").SemVerVersion'.
|
|
Lint
Process completed with exit code 2.
|
|
Lint:
app/scripts/controllers/swaps/swaps.test.ts#L1265
Some tests seem to be commented
|
|
Lint:
app/scripts/controllers/swaps/swaps.test.ts#L1286
Some tests seem to be commented
|
|
Lint:
app/scripts/controllers/swaps/swaps.test.ts#L1306
Some tests seem to be commented
|
|
Lint:
app/scripts/controllers/swaps/swaps.test.ts#L1390
Some tests seem to be commented
|
|
Lint:
app/scripts/controllers/swaps/swaps.test.ts#L1406
Some tests seem to be commented
|
|
Lint:
app/scripts/controllers/swaps/swaps.test.ts#L1429
Some tests seem to be commented
|
|
Lint:
test/e2e/flask/solana/common-solana.ts#L91
'#fb9f18' Hex color values are not allowed. Consider using design tokens instead. For support reach out to the design system team #metamask-design-system on Slack
|
|
Lint:
ui/components/app/alert-system/alert-modal/alert-modal.tsx#L209
React Hook "useI18nContext" is called conditionally. React Hooks must be called in the exact same order in every component render. Did you accidentally call a React Hook after an early return?
|
|
Lint:
ui/components/app/assets/asset-list/asset-list-control-bar/asset-list-control-bar.tsx#L83
React Hook useMemo has an unnecessary dependency: 'TEST_CHAINS'. Either exclude it or remove the dependency array. Outer scope values like 'TEST_CHAINS' aren't valid dependencies because mutating them doesn't re-render the component
|
|
Lint:
ui/components/app/assets/asset-list/asset-list-control-bar/asset-list-control-bar.tsx#L109
React Hook useEffect has missing dependencies: 'allOpts', 'currentNetwork.chainId', 'dispatch', and 'tokenNetworkFilter'. Either include them or remove the dependency array
|
Loading