feat: GitHub-hosted runners for benchmarks #58772
Annotations
10 warnings
|
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