`--update-data` introduces new blank lines when aligning multi-line expected info comments
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 20.6k
- Forks
- 3.3k
- Métricas de merge de PR
- Métricas de PR pendientes
Descripción
Bug Report
After using --update-data in https://github.com/python/mypy/pull/19904/, I'm seeing a lot of diffs like this:
[case testErrorCodeIgnoreMultiple1]
a = 'x'.foobar(b) # type: ignore[name-defined, attr-defined]
a = 'x'.foobar(b) # type: ignore[name-defined, xyz] # E: "str" has no attribute "foobar" [attr-defined] \
- # N: Error code "attr-defined" not covered by "type: ignore" comment
+ # N: Error code "attr-defined" not covered by "type: ignore" comment ignoring [name-defined, xyz]
a = 'x'.foobar(b) # type: ignore[xyz, w, attr-defined] # E: Name "b" is not defined [name-defined] \
- # N: Error code "name-defined" not covered by "type: ignore" comment
+ # N: Error code "name-defined" not covered by "type: ignore" comment ignoring [xyz, w, attr-defined]
+
+
it seems to apply extra whitespace at the end of the test case if it realigns a line continuation
Semi-related to https://github.com/python/mypy/issues/17465, but only a little, I think.
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.
Línea de trabajo
Comienza con el punto de entrada --update-data y el fixture testErrorCodeIgnoreMultiple1 mostrado en el informe. Reproduce la actualización y sigue la alineación de expected-info multilínea; se considera terminado cuando los comentarios realineados siguen siendo correctos sin añadir líneas en blanco al final del caso de prueba.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- testing-qa, tooling
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 45/100