apollographql / apollographql/fullstack-tutorial

Confusing naming of function passed to Array.map

Abierto
#138 0 comentarios 0 reacciones 0 asignados Ver en GitHub
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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.