Confusing naming of function passed to Array.map
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 1/5
- Tempo stimato
- Meno di un'ora
- Idoneità per principianti
- 55/100
- Tipo di issue
- Refactoring
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Ferma
- Stack tecnologico
- javascript
- Ambito
- backend
Direzione di ricerca
Apri final/server/src/datasources/launch.js alla definizione di launchReducer e verifica dove viene passato ad Array.map. Controlla ogni altro riferimento alla funzione, quindi aggiorna la denominazione in modo coerente affinché il nome della funzione rifletta il suo ruolo di mapping.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Hello,
not a real issue but just a naming of a method that confused me for a bit.
In datasource/launch there is method called launchReducer passed to Array.map. This method is responsible to take each launch returned by the Rest API and mapping it to another object.
So I guess this function should be called lauchMapper, because that's what it does. It doesn't reduce multiple element into a single one, isn't it?
Just highlighting this because it is a bit confusing
- Lingua principale
- TypeScript
- Stelle
- 1.2k
- Fork
- 808
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di apollographql/fullstack-tutorial
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
-
investigate
Difficoltà 4/5 3-5 giorni Idoneità per principianti 30/100
-
tutorial links are broken Apertadocumentation
apollographql/fullstack-tutorial#285 · 3 commenti · 1 assegnatario ·
-
investigate
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
apollographql/fullstack-tutorial#255 · 2 reazioni ·
-
enhancement
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
apollographql/fullstack-tutorial#254 · 2 commenti ·
Tutte le issue di apollographql/fullstack-tutorial
Issue simili
-
bug(cli): hapi doctor inline-media prints a fabricated B:\ helper-script path in packaged installs Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
Crush Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
catppuccin/catppuccin#3125 ·
-
Add a SECURITY.md Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
ElementsProject/cln-application#167 · 1 commento · 1 reazione ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
Quantco/pnpm-licenses#17 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100