Found unused variables in TypeScript code JS-0356
Performance
Major
a year ago
—
a year old
'logOut' is defined but never used. Allowed unused vars must match /^_/u
7import UpdateProfileForm, { AvatarUploadForm } from '@/components/forms/Profile.forms';
8import { Error, Info, Success, Warning } from '@/components/Feedback';
9import { SocialProfileForm } from '@/app/social/forms/SocialProfile.forms';
10import { checkTokenType, logOut } from '@/services/auth.services'; 11import {
12 checkProfileEmpty,
13 checkSocialProfileEmpty,'useNavigate' is defined but never used. Allowed unused vars must match /^_/u
1import React from 'react';
2import { Link, Navigate, Outlet, useNavigate } from 'react-router-dom'; 3import { useQuery, useQueryClient } from '@tanstack/react-query';
4import { toast } from 'react-hot-toast';
5import { DevCladSVG } from '@devclad/ui';
Description
Unused variables are generally considered a code smell and should be avoided.
Removing unused references - It prevents unused modules from being loaded at runtime, improving performance, and preventing the compiler from loading metadata that will never be used. - It prevents conflicts that may occur when trying to reference another variable.
NOTE: If you have intentionally left a variable unused, we suggest you to prefix the variable name with a _ to prevent them from being flagged by DeepSource.
Bad Practice
import fs from 'fs' // <- unused
import { readFileSync } from 'fs'
const text = readFileSync('declaration_of_independence.txt', 'utf-8')
console.log(text)
Recommended
import { readFileSync } from 'fs'
const text = readFileSync('declaration_of_independence.txt', 'utf-8')
console.log(text)