Empty line before comment should (sometimes) be preserved
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 48/100
Línea de trabajo
Comienza con el tratamiento que hace zeek-format de las líneas vacías antes de los comentarios menores, comparando los comentarios posteriores a un evento completado con los comentarios que son el primer nodo después de {. Confirma que formatear el ejemplo conserva la línea en blanco y que volver a formatear el resultado no lo modifica, mientras que las líneas vacías inmediatamente después de { siguen eliminándose.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Currently zeek-format removes empty lines before any minor comments, e.g.,
event new_event(name: string, params: call_argument_vector)
{
print name;
}
# Should preserve empty line before.
first gets formatted to
event new_event(name: string, params: call_argument_vector)
{
print name;
}
# Should preserve empty line.
and then again (breaking idempotency) to
event new_event(name: string, params: call_argument_vector)
{
print name;
}
# Should preserve empty line.
I would have expect this to not change formatting at all.
zeek-format should still remove empty after e.g., { (comment first node in block).
Labelling as bug due to idempotency breaking.
- Lenguaje dominante
- Python
- Estrellas
- 11
- Forks
- 4
- Merge medio
- 3 d 19 h
- PR fusionados (30 d)
- 1
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
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 45/100
zeek/zeekscript#137 ·
-
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 45/100
zeek/zeekscript#83 · 1 comentario ·
-
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
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
stephrobert/dsoxlab#238 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
sublimehq/package_control#1780 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
nwg-piotr/nwg-displays#145 ·