include relevant parts of logs in build/test comments
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 55/100
Línea de trabajo
Start by reading the implementation in pull request #3720 and trace how reviewbot publishes build results and links to task logs in Phabricator and GitHub comments. Find how Treeherder selects relevant log errors, then determine how to include them without delaying publication or making comments too large. Done means comments show useful build errors promptly and the approach can also support test logs later.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
https://github.com/mozilla/code-review/pull/3720 has implemented support for running builds as part of reviewbot. At the moment, the comments it posts to phabricator and github only include the names of the tasks that failed, and links to their logs. It would be helpful to include the relevant errors from the logs, similar to how treeherder does it.
We'll need to do this in a way that doesn't cause publication to be greatly delayed (fast results are important) or comments to get too large. Having collapseable sections for them might be useful, if possible. We should also design this in a way such that we can do a similar thing for test logs when we start running tests here.
- Lenguaje dominante
- Python
- Estrellas
- 68
- Forks
- 53
- Merge medio
- 1 d 13 h
- PR fusionados (30 d)
- 52
Preparar el entorno
- Incluye un Dockerfile o un 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 mozilla/code-review
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
mozilla/code-review#3399 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
mozilla/code-review#3740 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
mozilla/code-review#3738 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
mozilla/code-review#3737 ·
Los mantenedores suelen responder en 1 día
-
Enable before/after by defaultAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
mozilla/code-review#3710 ·
Los mantenedores suelen responder en 1 día
Todos los issues de mozilla/code-review
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
mohitmishra786/readprism#322 ·
Los mantenedores suelen responder en 1 día
-
good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
API-key credentials accept a blank key: empty via the API, spaces-only via the builder's Add API key dialogPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
Significant-Gravitas/AutoGPT#15727 ·
Los mantenedores suelen responder en 1 día