sometimes merges comments where should not
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
Línea de trabajo
Reproduce el problema con la entrada bug.zeek proporcionada y el punto de entrada zeek-format, y luego sigue cómo se analizan y emiten las líneas cortas y los comentarios ##. Compara la salida del formateador con el límite de comentario esperado por el analizador de Zeek; se considera terminado cuando las dos directivas @load permanecen separadas y el comportamiento está cubierto por una prueba de regresión.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
looks like for short lines ## comments are pulled up, with no break, which doesnt work where the zeek parser is greedy
(or alternately i guess this is a zeek bug that the ## should have started a new comment...)
✦4 ❯ cat bug.zeek
@load ./foo.zeek
## @load ./bar.zeek
✦4 ❯ zeek-format bug.zeek
@load ./foo.zeek## @load ./bar.zeek
- Lenguaje dominante
- Python
- Estrellas
- 11
- Forks
- 4
- Merge medio
- 1 d 20 h
- PR fusionados (30 d)
- 1
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 zeek/zeekscript
-
Dificultad 3/5 1-2 días Aptitud para principiantes 38/100
zeek/zeekscript#120 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
zeek/zeekscript#91 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
zeek/zeekscript#83 · 1 comentario ·
-
Function calls incorrectly formattedQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
zeek/zeekscript#69 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
zeek/zeekscript#63 ·
Todos los issues de zeek/zeekscript
Issues similares
-
Path subtraction removes repeated suffix characters instead of one suffixPosiblemente ocupada @Gwerdonatus la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
joke2k/django-environ#671 ·
-
good first issue help wanted windows
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
-
test_pgbouncer deploys the server under test from Charmhub instead of the local build (track/1.31)Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
canonical/temporal-k8s-operator#195 · 1 comentario ·
Los mantenedores suelen responder en 4 días
-
`burr` fails to start when the Python path has a space in itPosiblemente ocupada @enekoreto la tomó hoy. Abiertostatus/needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 5 días
-
device support request
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
zigpy/zha-device-handlers#5461 ·
Los mantenedores suelen responder en 4 días