Duplicated code
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 42/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- api
Línea de trabajo
Empieza leyendo dom.js en las líneas 27-56 y las pautas enlazadas sobre duplicación. Refactoriza el código duplicado en ese rango sin cambiar el comportamiento; se considera terminado cuando la lógica repetida está consolidada y el comportamiento existente sigue cubierto por las pruebas del repositorio.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
See here: https://github.com/ryanmcdermott/clean-code-javascript#remove-duplicate-code for explanation on why it is bad to duplicate code.
The code here: https://github.com/fac-14/NDJS-API/blob/master/dom.js#L27-L56 has some dupliation and can be refactored.
(Sorry, I know you guys had a lot to do in a short period of time).
- Lenguaje dominante
- JavaScript
- Estrellas
- 0
- Forks
- 0
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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.
Más de fac-14/NDJS-API
-
README needs contentAbierto
Dificultad 1/5 1-3 horas Aptitud para principiantes 82/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
-
Readability 📖Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 48/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 35/100
-
🚨Use of innerHTMLAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 35/100
Todos los issues de fac-14/NDJS-API
Issues similares
-
[BUG]: Prompt compressor crashes with TypeError when a past chat has a non-string response.textPosiblemente ocupada @ManojDevarajulu la tomó hoy. Abiertopossible bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Mintplex-Labs/anything-llm#6737 ·
Los mantenedores suelen responder en 1 día
-
Add: Tooncast brazil feedAbiertocheck:passed feeds:add
Dificultad 2/5 Menos de una hora Aptitud para principiantes 62/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
motiondivision/motion#3935 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
codex-team/editor.js#3037 ·
-
No Code Attached Yet
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
joomla/joomla-cms#48626 ·
Los mantenedores suelen responder en 1 día