sindresorhus/eslint-plugin-unicorn

More powerful `new-for-builtins`

Chiusa

#1835 aperta il 27 mag 2022

Vedi su GitHubGuida contributor
 (5 commenti) (2 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

Currently, new-for-builtins enforce builtins used either call or new.

My idea is to extend this rule to check all builtins.

  • Objects like JSON/Math/Reflect/Temporal should be neither call nor new
  • Objects from global objects like Intl.DateTimeFormat/Temporal.Now... should also be checked
  • Only fix cases new and call result the same Function('')/new Function(''), Error('')/new Error(''), use suggestion for other cases like new Symbol()(runtime error)

Guida contributor

Direzione di ricerca
Esamina l'implementazione esistente della regola `new for builtins` nel file sorgente (probabilmente `rules/new for builtins.js`) per comprenderne la logica attuale di rilevamento dei built in. Quindi estendila per controllare anche oggetti come `JSON`, `Math`, `Reflect`, `Temporal` e metodi di oggetti globali come `Intl.DateTimeFormat`. Assicurati che la regola suggerisca correzioni appropriate nei casi in cui `new` e `call` sono equivalenti ed eviti falsi positivi.
Tech stack
javascriptnodejs
Dominio
developer experiencetooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ESLintJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60