palantir/blueprint

ESLint deprecated component usage detector does not handle aliased imports

Chiusa

#6408 aperta il 26 set 2023

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)TypeScript (2167 fork)batch import
Domain: toolingP2Type: bughelp wanted

Metriche repository

Star
 (20.263 stelle)
Metriche merge PR
 (Merge medio 43g 10h) (27 PR mergiate in 30 g)

Descrizione

Environment

  • Package version(s): @blueprintjs/eslint-plugin: 5.0.0
  • Operating System: Windows 10/11

Steps to reproduce

  1. Use an aliased import of a deprecated BlueprintJS component somewhere:

import { DateInput as BlueprintDateInput } from "@blueprintjs/datetime";

  1. Use "plugin:@blueprintjs/recommended" in your eslintrc file.

Actual behavior

No errors detected.

Expected behavior

A message which tells that the usage of DateInput is deprecated.

  71:6  error  Usage of DateInput is deprecated, migrate to DateInput3 instead  @blueprintjs/no-deprecated-components

The error is properly displayed if we change the aliased import to a non-aliased import.

Guida contributor

Direzione di ricerca
Indaga l'implementazione della regola ESLint in @blueprintjs/eslint plugin, probabilmente nella directory src/rules/no deprecated components. La regola deve gestire le importazioni con alias controllando i nodi ImportSpecifier in cui il nome importato corrisponde ai componenti deprecati, indipendentemente dall'alias locale. Cerca la funzione che elabora ImportDeclaration e modificarla per catturare anche le importazioni con alias. Considera di testare con casi come 'import { DateInput as BlueprintDateInput }'.
Tech stack
typescriptreactjavascript
Dominio
frontendtooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Node.jsESLint
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60