Found unused variables in TypeScript code JS-0356
Performance
Major
2 years ago
—
2 years old
'Subject' is defined but never used
140
141declare global {
142 namespace Cypress {
143 interface Chainable<Subject> {144 getOauth2Data(): Cypress.Chainable;
145 oauthLogin(oauth2Data: any, username: string, password: string): Cypress.Chainable;
146 keycloakLogin(oauth2Data: any, username: string, password: string): Cypress.Chainable;'entityItemSelector' is defined but never used
1import { entityItemSelector } from '../../support/commands'; 2import {
3 entityTableSelector,
4 entityDetailsButtonSelector,'entityItemSelector' is defined but never used
1import { entityItemSelector } from '../../support/commands'; 2import {
3 entityTableSelector,
4 entityDetailsButtonSelector,'entityItemSelector' is defined but never used
1import { entityItemSelector } from '../../support/commands'; 2import {
3 entityTableSelector,
4 entityDetailsButtonSelector,'Authority' is defined but never used
1import { NgModule } from '@angular/core';
2import { RouterModule } from '@angular/router';
3import { Authority } from '../config/authority.constants'; 4
5@NgModule({
6 imports: [
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)