apollographql / apollographql/fullstack-tutorial
Confusing naming of function passed to Array.map
- Lenguaje dominante
- TypeScript
- Estrellas
- 1.2k
- Forks
- 808
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
Hello,
not a real issue but just a naming of a method that confused me for a bit.
In [datasource/launch](https://github.com/apollographql/fullstack-tutorial/blob/master/final/server/src/datasources/launch.js#L10) 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
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Línea de trabajo
Abre final/server/src/datasources/launch.js en la definición de launchReducer e inspecciona dónde se pasa a Array.map. Comprueba cualquier otra referencia a la función y, a continuación, actualiza la nomenclatura de forma coherente para que el nombre de la función refleje su función de mapeo.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript
- Área
- backend
- Tipo de issue
- Refactorización
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 55/100