Add auto-fix for `no-array-reduce`

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 foo = array.reduce(callback, initValue);

let foo = initValue;
for (const [index, element] of array.entries()) {
  foo  = callback(foo, element, index, array);

  // Or safer
  // foo  = callback.call(array, foo, element, index, array); 
}

 const foo = array.reduce(callback); // No inital value

let foo;
for (const [index, element] of array.entries()) {
  if (index === 0) {
   foo = element;
   continue;
  }
  foo  = callback(foo, element, index, array);
}

 const foo = array.reduce((accumulator, element, index, array) => {
	return doThingsWith(accumulator, element, index, array)
}, initValue);

let foo = initValue;
for (const [index, element] of array.entries()) {
  foo  = doThingsWith(foo, element, index, array);
}

Guida contributor

Direzione di ricerca

Rivedi l'implementazione della regola esistente nel file `no array reduce.js` nel repository. Il problema fornisce un chiaro codice prima/dopo per tre casi. Usa l'API fix di ESLint per sostituire le chiamate `reduce` con cicli `for`. Controlla i commenti del problema per eventuali considerazioni o limitazioni aggiuntive.

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.

Attiva

ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.

Chiara

Prerequisiti

JavaScriptESLintGit

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

60