Skip to content

fix: abacus input types should always be string #4712

fix: abacus input types should always be string

fix: abacus input types should always be string #4712

Triggered via pull request October 25, 2024 19:55
Status Success
Total duration 3m 14s
Artifacts

commitlint.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

11 warnings
lint
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, dydxprotocol/setup-pnpm@v2.0.0, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
lint: src/components/Button.tsx#L228
Unnecessary conditional, value is always truthy
lint: src/components/ComboboxMenu.tsx#L194
Unnecessary conditional, expected left-hand side of `??` operator to be possibly null or undefined
lint: src/components/Details.tsx#L224
Unnecessary conditional, value is always truthy
lint: src/components/Details.tsx#L232
Unnecessary conditional, value is always truthy
lint: src/components/Details.tsx#L244
Unnecessary conditional, value is always truthy
lint: src/components/GuardedMobileRoute.tsx#L27
React Hook useEffect has missing dependencies: 'canAccountTrade', 'dispatch', and 'isTablet'. Either include them or remove the dependency array
lint: src/components/GuardedMobileRoute.tsx#L38
React Hook useEffect has a missing dependency: 'navigate'. Either include it or remove the dependency array
lint: src/components/QrCode.tsx#L73
React Hook useEffect has a missing dependency: 'qrCode'. Either include it or remove the dependency array
lint: src/components/QrCode.tsx#L77
React Hook useEffect has a missing dependency: 'size'. Either include it or remove the dependency array. Mutable values like 'ref.current' aren't valid dependencies because mutating them doesn't re-render the component
lint: src/components/QrCode.tsx#L85
React Hook useEffect has a missing dependency: 'qrCode'. Either include it or remove the dependency array