Extraneous space inserted after trailing comma
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
Comienza localizando los puntos de entrada del formatter para las construcciones enum, set-like, record y vector; después, inspecciona cómo se representan las comas finales. Reproduce los ejemplos del issue y compara cada salida con el comportamiento de enum. Se considera terminado cuando las comas finales se conservan sin un espacio adicional cuando no existe un argumento posterior, con cobertura para las construcciones indicadas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
In many places Zeek allows trailing commas. This seems to be handled inconsistently.
For enums I see expected behavior:
type E: enum { a, b, c, };
For set-like constructs I see an additional space:
const s = { 1, 2, 3, };
# ^
const r: RecordType = [ $a=1, $b=2, $c=3, ];
# ^
We probably have the same issue for function-like constructors, e.g.,
const a = vector(1, 2, 3, );
# ^
The issue seems to be that we insert an space after , in the contexts even though it is not followed by more arguments.
- 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 48/100
zeek/zeekscript#91 ·
-
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 ·
Todos los issues de zeek/zeekscript
Issues similares
-
agent-ready documentation needs-triage
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
-
documentation
Dificultad 1/5 Menos de una hora Aptitud para principiantes 91/100
-
workflow-status page template still says reusable workflows are "triggered only by workflow_call:" Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
-
Add https://search.jeremyh.xyz/ Abiertoinstance instance add
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
searxng/searx-instances#939 · 1 comentario ·
-
area-deployment area-integrations triage:bot-seen
Dificultad 2/5 Medio día Aptitud para principiantes 86/100