Don't use javascript reserved word as Object keys
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- backend
Línea de trabajo
Comienza en lib/blocks.js alrededor de la línea 223 e inspecciona las claves de objeto que usan palabras reservadas de JavaScript. Comprueba si las claves afectadas pueden usar el nombre sugerido o el acceso entre comillas sin cambiar el comportamiento del parser. El trabajo estará terminado cuando las claves con palabras reservadas de esta zona estén resueltas y las comprobaciones existentes del proyecto pasen correctamente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Hi,
I have observed the block.js file makes use of javascript keywords as objects keys. Ref: here.
Although this isn't prohibited, it isn't the correct thing to do.
- Editors highlight is wrongly
- Code minifiers have trouble with them.
Its best to not use them at all, in this scenarios it can be named as proceed. i.e the function returns. If that's not possible, at least you can quote them as below.
from
var object = {
continue: function(){}
}
object.continue();
to
var object = {
'continue': function(){}
}
object['continue']()
Please let me know how you wish to address this, I'll submit a PR with the changes.
- Lenguaje dominante
- JavaScript
- Estrellas
- 1.6k
- Forks
- 231
- Merge medio
- 2 d 17 h
- PR fusionados (30 d)
- 2
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 commonmark/commonmark.js
-
White space handling looks inconsistent between HTML block element vs HTML inline elementPosiblemente ocupada @mamicicekel la tomó hace 3 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
commonmark/commonmark.js#303 · 2 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
commonmark/commonmark.js#298 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
commonmark/commonmark.js#287 · 1 reacción ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 20/100
commonmark/commonmark.js#275 ·
-
Delegate node walking to othersAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
commonmark/commonmark.js#267 · 3 comentarios ·
Todos los issues de commonmark/commonmark.js
Issues similares
-
[BUG] Multi-day events show "Ended" while still in progressPosiblemente ocupada @tarunagnihotri534 la tomó hoy. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
data-umbrella/du-event-board#231 · 2 comentarios ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
NaturalIntelligence/fast-xml-parser#888 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 80/100
USACE/chart-docs#766 ·
-
bug callouts regression revealjs
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
quarto-dev/quarto-cli#15014 ·
Los mantenedores suelen responder en 1 día