Confusing Variable Naming: "step" & "move"
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 11.8k
- Forks
- 7.9k
- Merge medio
- 1 d 11 h
- PR fusionados (30 d)
- 11
Descripción
At line 1139, jumpTo() requires an argument named step.
However, on lines 1017, and 1106, it passes the variable, move, into jumpTo function, which is confusing.
I think it's better either change the naming of the argument of jumpTo() (:1139),
or switch the naming of currentValue and index of map() (:1017, :1106)
The below PR goes the second way since I think move is more like representing the "movement", on the other hand, step is understood as a "number" in most cases.
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza inspeccionando jumpTo() en la línea 1139 y las llamadas a map() en las líneas 1017 y 1106. Confirma qué nombres se utilizan en cada ubicación, luego haz que la nomenclatura sea coherente con el significado previsto y verifica que el código afectado siga siendo claro de leer.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript
- Área
- documentation
- Tipo de issue
- Refactorización
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 38/100