Found unused variables in TypeScript code JS-0356
Performance
Major
4 months ago
—
8 months old
'EditIntegrationForm' is assigned a value but never used
23 integration: RouterOutputs["integration"]["byId"];
24}
25
26export const EditIntegrationForm = ({ integration }: EditIntegrationForm) => { 27 const t = useI18n();
28 const { openConfirmModal } = useConfirmModal();
29 const secretsKinds ='UserProfileAvatarForm' is assigned a value but never used
17 user: RouterOutputs["user"]["getById"];
18}
19
20export const UserProfileAvatarForm = ({ user }: UserProfileAvatarForm) => { 21 const { mutate } = clientApi.user.setProfileImage.useMutation();
22 const [opened, { toggle }] = useDisclosure(false);
23 const { openConfirmModal } = useConfirmModal();
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)