Replace `var` with `let` and `const`
@gitdevjin ya está trabajando en esto.
Desde el 2/10/2024.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- backend
Línea de trabajo
Comienza revisando index.js y README.md para localizar cada uso de var e inspeccionar su ámbito y uso. Reemplaza las declaraciones por let o const según corresponda y, después, verifica que ambos archivos no contengan declaraciones var restantes y que los ejemplos documentados sigan coincidiendo con el código.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Description
According to the docs, using var is not recommended, unless it's necessary, It would be useful to replace var in index.js with let or const depending on their scope and usage. And also as it's less error-prone, this change can make the code more robust for the possible future code change.
Task
- Replace
varwithletorconstinindex.jsfile - Update
varwithletorconstinREADME.mdfile
Supporting Docs and Articles
- Lenguaje dominante
- JavaScript
- Estrellas
- 2k
- Forks
- 235
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la 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.
Más de expressjs/cookie-parser
-
question
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
expressjs/cookie-parser#168 · 5 comentarios ·
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
expressjs/cookie-parser#138 ·
-
req.cookies is showing nullPosiblemente ocupada @D-393Patel la tomó hace 334 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
expressjs/cookie-parser#128 ·
-
perf: limit cookie parse only on some cookiePosiblemente ocupada @cesco69 la tomó hace 545 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 42/100
expressjs/cookie-parser#126 · 1 reacción ·
-
Request for ESM SupportAbiertofuture question
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
expressjs/cookie-parser#124 · 7 reacciones ·
Todos los issues de expressjs/cookie-parser
Issues similares
-
[Bug] Dev image build fails: `npm install` in the root Dockerfile runs out of memory (exit 134)Posiblemente ocupada @andest01 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
good first issue hacktoberfest help wanted
Dificultad 2/5 Menos de una hora Aptitud para principiantes 76/100
pacocartones/free-llm-api-hub#262 ·
Los mantenedores suelen responder en 1 día
-
agent-task risk:low
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
guyellis/http-status-check#512 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
microsoft/playwright#43327 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
microsoft/TypeScript#64758 ·
Los mantenedores suelen responder en 1 día