Rule proposal: `prefer-math-abs`

Metriche repository

Star

5022 stelle (5022 stelle)

Metriche merge PR

Merge medio 1g 16h (Merge medio 1g 16h)399 PR mergiate in 30 g (399 PR mergiate in 30 g)

Descrizione

const a = b < 0 ? -b : b;
const a = b <= 0 ? -b : b;
const a = b < 0 ? 0 - b : b;
const a = b <= 0 ? 0 - b : b;

const a = b > 0 ? b : -b;
const a = b >= 0 ? b : -b;

const a = b < 0 ? b * -1 : b;

if (number > POSITIVE_CONSTANT || number < -POSITIVE_CONSTANT) {}

const a = Math.abs(b);

if (Math.abs(number) > POSITIVE_CONSTANT) {}

Guida contributor

Direzione di ricerca

Studia le regole esistenti nel repository `eslint plugin unicorn` (ad esempio, la directory `rules/`) per comprendere lo schema per creare una nuova regola. Implementa la regola per rilevare espressioni condizionali che possono essere sostituite da `Math.abs()`. Scrivi i test in `test/` e aggiorna la documentazione in `docs/rules/`. C'è un commento nell'issue che potrebbe fornire ulteriori indicazioni, ma non sono visibili PR collegate o domande dei maintainer.

Tech stack

javascript

Dominio

tooling

Tipo issue

Funzionalità

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

JavaScript

Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.

70