sindresorhus/eslint-plugin-unicorn

`prevent-abbreviations`: Smarter `whitelist` option possibly deprecating `ignore` option

Chiusa

#930 aperta il 9 dic 2020

Vedi su GitHubGuida contributor
 (1 commento) (6 reazioni) (0 assegnatari)JavaScript (468 fork)user submission
enhancementhelp wanted

Metriche repository

Star
 (5022 stelle)
Metriche merge PR
 (Merge medio 1g 16h) (399 PR mergiate in 30 g)

Descrizione

There are now two options to exclude an identifier from linting: older whitelist and newer ignore. I think whitelist alone would suffice with the following modification.

We could make whitelist match any subsequence of the split identifier. For example (following up this https://github.com/sindresorhus/eslint-plugin-unicorn/issues/567#issuecomment-591007369): For fooE2eBar all of these would be checked against whitelist:

  • foo
  • e
  • 2
  • bar
  • barE
  • e2
  • 2e
  • eBar
  • fooE2
  • e2e
  • 2eBar
  • fooE2e
  • e2eBar
  • fooE2eBar

And if any of them is in the whitelist that subsequence is ignored.

But in my opinion it should not ignore the whole identifier if only subsequence matches whitelist.

For example, if the user had e2e in their whitelist, e2eErr should still be fixed to e2eError.

From https://github.com/sindresorhus/eslint-plugin-unicorn/pull/891#issuecomment-713729469

Guida contributor

Direzione di ricerca
Indaga l'implementazione attuale della regola prevent abbreviations, in particolare le opzioni whitelist e ignore. Rivedi la PR collegata #891 e il commento #713729469 per comprendere le discussioni precedenti. Valuta se l'approccio proposto di corrispondenza delle sottosequenze sia fattibile e se possa sostituire l'opzione ignore. Considera casi limite come sottosequenze sovrapposte e l'impatto sulle prestazioni. Quindi proponi un design e implementa la modifica.
Tech stack
javascript
Dominio
tooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
4
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Knowledge of ESLint plugin developmentUnderstanding of prevent abbreviations ruleFamiliarity with Node.js and JavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30