`--update-data` introduces new blank lines when aligning multi-line expected info comments
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 20.6k
- Forks
- 3.3k
- Métriques de merge des PR
- Métriques de PR en attente
Description
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.
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par le point d’entrée --update-data et le fixture testErrorCodeIgnoreMultiple1 présenté dans le rapport. Reproduisez la mise à jour et suivez l’alignement de expected-info sur plusieurs lignes ; le travail est terminé lorsque les commentaires réalignés restent corrects sans ajouter de lignes vierges à la fin du cas de test.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- testing-qa, tooling
- Type d'issue
- Bug
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 45/100