sindresorhus/eslint-plugin-unicorn

Rule proposal: `prefer-number-is-integer`

Chiusa

#1460 aperta il 1 ago 2021

Vedi su GitHubGuida contributor
 (3 commenti) (4 reazioni) (0 assegnatari)JavaScript (468 fork)user submission
help wantednew rule

Metriche repository

Star
 (5022 stelle)
Metriche merge PR
 (Merge medio 4h 30m) (26 PR mergiate in 30 g)

Descrizione

Use Number.isInteger. Not really sure if there's any benefit to using it. Maybe it's more performant. Makes the code marginally cleaner.

Fail

number % 1 === 0;

Pass

Number.isInteger(number);

Guida contributor

Direzione di ricerca
Esamina il repository eslint plugin unicorn per comprendere il modello di creazione delle regole. Guarda le regole esistenti come `prefer number is nan` per una guida. Nei commenti dell'issue, i manutentori potrebbero aver discusso i vantaggi prestazionali; nota che la proposta manca di una forte motivazione. Considera di ricercare se `Number.isInteger` è effettivamente più veloce di `number % 1 === 0`, e verifica se la regola può essere implementata utilizzando il nodo AST `BinaryExpression` con l'operatore `%`.
Tech stack
javascript
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.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Familiarity with ESLint rule structureJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60