Backend issue lists report in-patch warnings and out-of-patch errors as not publishable
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 75/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Área
- backend-api-design
Línea de trabajo
Empieza por localizar las anotaciones publishable en IssueViewSet y IssueCheckDetails, y luego compáralas con la propiedad IssueLink.publishable y las reglas compartidas de publishable descritas en el issue. Comprueba el endpoint de diff issues y el filtro de detalles de checks para las advertencias dentro del parche y los errores fuera de él. El trabajo estará terminado cuando ambos endpoints indiquen que esos casos son publishable y el filtro predeterminado de detalles de checks siga siendo coherente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
IssueLink.publishable treats an issue as publishable when it is in the patch or is an error (in_patch is True or level == "error"). The bulk creation response, the publishable filter of the check details endpoint, and the publishable counters for diffs and check stats use the same rule.
Two endpoints compute the publishable field with & instead of |:
GET /v1/diff/<diff_provider_id>/issues/(IssueViewSet)GET /v1/check/<repository>/<analyzer>/<check>/(IssueCheckDetails)
.annotate(publishable=Q(issue_links__in_patch=True) & Q(level=LEVEL_ERROR))
As a result, these endpoints return:
publishable: falsefor warnings inside the patch.publishable: falsefor errors outside the patch, andnullwhenin_patchis unknown.
On the check details endpoint, the default ?publishable=true filter keeps these issues while each one reports publishable: false.
The annotation came in with #2451, which moved publishable to IssueLink. The model property kept the or rule.
Steps to reproduce
- Link an error issue to a diff with
in_patch=False, and a warning issue within_patch=True. - Request
/v1/diff/<diff_provider_id>/issues/.
Expected: both issues have publishable: true.
Actual: both issues have publishable: false.
- Lenguaje dominante
- Python
- Estrellas
- 68
- Forks
- 54
- Merge medio
- 2 d 4 h
- PR fusionados (30 d)
- 24
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
-
ensure results are returned even when `code-review` task doesn't run automaticallyPosiblemente ocupada @bhearsum la tomó hace 14 días. Abierto
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 55/100
mozilla/code-review#3739 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
use one try push for linting and build/testPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
mozilla/code-review#3738 ·
Los mantenedores suelen responder en 1 día
-
support performance profiles in tasksPosiblemente ocupada @La0 la tomó hace 15 días. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
mozilla/code-review#3737 · 3 comentarios ·
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
-
Server mode OAuth2: 'NoneType' object is not subscriptable on every request after re-submitting the OAuth2 login while already logged inPosiblemente ocupada @Rarasinhoo la tomó hoy. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
pgadmin-org/pgadmin4#10733 ·
Los mantenedores suelen responder en 3 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
aws-samples/sample-per-model-bedrock#752 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
int-brain-lab/mpci#32 ·
-
docs(api): add sesmax-core as a layer below radtrax and sesmax-ocean in the dependency graphAbiertoarea:docs priority:p2 type:docs
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
jejjohnson/geostack#120 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
sublimelsp/LSP-vue#203 · 1 reacción ·
Los mantenedores suelen responder en 1 día