Found trailing undefined in function call JS-W1042
Anti-pattern
Minor
6 months ago
—
7 months old
Remove redundant
undefined from function call 2import clsx from 'clsx';
3import {useDocsSidebar} from '@docusaurus/theme-common/internal';
4import styles from './styles.module.css';
5export const HiddenSidebarContainerContext = React.createContext(undefined); 6
7export default function DocPageLayoutMain({ hiddenSidebarContainer, children }) {
8 const sidebar = useDocsSidebar();
Description
When an argument is omitted from a function call, it will default to undefined.
It is therefore redundant to explicitly pass an undefined literal as the last argument.
Bad Practice
function hasOptionalParam(a: number, b?: number) {
// ...
}
hasOptionalParam(1, undefined)
Recommended
function hasOptionalParam(a: number, b?: number) {
// ...
}
hasOptionalParam(1)
hasOptionalParam(1, 2)