psf/black

List concatenation, multiple lines

Open

#260 ouverte le 26 mai 2018

Voir sur GitHub
 (2 commentaires) (3 réactions) (0 assignés)Python (2 693 forks)batch import
F: linebreakF: symmetryT: enhancementT: stylehelp wanted

Métriques du dépôt

Stars
 (41 258 stars)
Métriques de merge PR
 (Merge moyen 8j 13h) (33 PRs mergées en 30 j)

Description

Probably not a bug, but I was wondering if the code below shouldn't be formatted more like call chains are. I did not expect black to change anything other than quotes in this example. Aligned brackets feel much more readable.

# before
-    search_fields = (
-        ['file__%s' % field for field in FileAdmin.search_fields] +
-        ['resource__%s' % field for field in ResourceAdmin.search_fields]
-    )
# black output
+    search_fields = ["file__%s" % field for field in FileAdmin.search_fields] + [
+        "resource__%s" % field for field in ResourceAdmin.search_fields
+    ]

Operating system: OSX 10.11.6 Python version: 3.6.5 Black version: 18.5b0 Does also happen on master: yes

Guide contributeur

Direction de recherche
Cette demande propose que la concaténation de listes sur plusieurs lignes soit formatée de manière similaire aux chaînes d'appels. Étudiez comment Black gère actuellement les expressions binaires impliquant des list comprehensions, en particulier dans le module blib2to3 ou lines. Envisagez d'ajuster la logique de division pour que l'opérateur + soit placé au début de la ligne de continuation, en alignement avec le style utilisé pour les appels chaînés. Testez l'exemple fourni pour vous assurer que la sortie correspond au formatage proposé.
Stack technique
python
Domaine
tooling
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
4
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Python 3.6+Familiarity with Black's codebase
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
35