Would be nice if there was an iterable interface to the NodeWalker
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 48/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- api
Línea de trabajo
Comienza por la implementación de NodeWalker devuelta por .walker() e inspecciona el comportamiento existente de next(). Añade compatibilidad con iterables para que for...of produzca los mismos eventos del walker y termine cuando se complete el recorrido; verifica el comportamiento con las pruebas existentes del proyecto, aunque el issue no nombra ningún archivo de pruebas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Right now the NodeWalker instance returned by .walker() is not an actual javascript iterable.
Would be great to be able to do:
const walker = ast.walker()
for(event of walker) {
// stuff here
}
But right now that doesn't work.
It should be pretty simple to add, the following works when I already have a walker instance so just adding that before it is returned will work...
walker[Symbol.iterator] = function() {
return {
next() {
const data = walker.next();
if (data) {
return {
value: data
};
}
return {done: true};
}
};
};
- Lenguaje dominante
- JavaScript
- Estrellas
- 1.6k
- Forks
- 234
- Merge medio
- 1 d 27 min
- PR fusionados (30 d)
- 1
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 15 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 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 ·
-
Inconsistent escape of image altAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 42/100
commonmark/commonmark.js#264 · 1 reacción ·
Todos los issues de commonmark/commonmark.js
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
bluerobotics/Cockpit-docs#48 ·
-
README is out of date: broken badge, wrong command name, wrong Node version, placeholder URLsAbiertodocumentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
PhyberApex/slds-json-schema-renderer#372 ·
Los mantenedores suelen responder en 1 día
-
usage: a custom provider's balance meters ("…%" in the balance path) ignore the Left/Used switchAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
[BUG] rag: sentence content can cross the reported sentence boundariesPosiblemente ocupada @LJK-ikun la tomó hoy. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día