Mozlint publisher ignores lineoffset, so multi-line warnings aren't posted inline
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 74/100
Línea de trabajo
Comienza en bot/code_review_bot/tasks/lint.py, en el nb_lines=1 codificado, y sigue cómo se analiza el artefacto JSON de mozlint. Revisa la definición de result.py de mozlint referenciada y el uso de lineoffset en compare-locales. Se considera terminado cuando las advertencias multilínea estén ancladas en todas las líneas afectadas y se publiquen inline cuando las líneas relevantes estén en el diff.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
I have updated compare-locales to provide a lineoffset on top of lineno when finding an error in a multiline string. Turns out that doesn't do anything.
The mozlint parser in hardcodes nb_lines=1 and never reads the lineoffset field from the mozlint JSON artifact (guessing it should use lineoffset + 1).
As a result, warnings that apply to a multi-line entity are anchored to a single line and dropped from publishing when that line isn't part of the diff.
Example revision: https://phabricator.services.mozilla.com/D304024 — 4 changed strings, only a general comment, no inline comments.
P.S. compare-locales still has an issue, because I set lineoffset as a value, not as a tuple. I honestly don't understand why that's the case, since lineno is already defined.
- Lenguaje dominante
- Python
- Estrellas
- 67
- Forks
- 52
- Merge medio
- 3 d 23 h
- PR fusionados (30 d)
- 54
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 45/100
mozilla/code-review#3666 ·
-
Remove the frontend Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
mozilla/code-review#3648 · 3 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
mozilla/code-review#3640 ·
-
mozilla/code-review#3638 · 1 asignado ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
mozilla/code-review#3530 · 1 comentario ·
Todos los issues de mozilla/code-review
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100