Sanity check: Ensure AST hasn't changed
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
Línea de trabajo
Empieza leyendo la documentación enlazada de Black sobre la comparación de ASTs antes y después del formateo; después, inspecciona los puntos de entrada de parser, analyzer y formatter de zeekscript. El issue no menciona archivos ni tests, por lo que completarlo requeriría definir el enfoque de comparación y añadir cobertura que demuestre que el formateo preserva la semántica.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Black has some interesting thoughts about checking that the AST hasn't changed after re-formatting to ensure that the code is semantically equivalent: https://black.readthedocs.io/en/stable/the_black_code_style/current_style.html#ast-before-and-after-formatting
Reformatting shouldn't introduce semantic changes, so I think this would be more of a double-check on the correctness of zeekscript itself.
- 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 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 ·
-
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 ·
Todos los issues de zeek/zeekscript
Issues similares
-
documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
ianktoo/project-monitor#14 ·
-
A wrong descriptor checksum is refused with the right one, also for an xprv or WIF descriptorAbiertosecurity
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
btclib-org/btclib-wallet#478 ·
Los mantenedores suelen responder en 1 día
-
security
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
btclib-org/btclib#2727 ·
Los mantenedores suelen responder en 1 día
-
good first issue lead variant:container
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
radialmonster/orf-cleanroom#10 ·
Los mantenedores suelen responder en 1 día
-
gdsa opencode: keep-alive cleanup trap fails with 'keepalive_pid: unbound variable'Posiblemente ocupada @darribas la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100