psf/black

List concatenation, multiple lines

Open

#260 geöffnet am 26. Mai 2018

Auf GitHub ansehen
 (2 Kommentare) (3 Reaktionen) (0 zugewiesene Personen)Python (2.693 Forks)batch import
F: linebreakF: symmetryT: enhancementT: stylehelp wanted

Repository-Metriken

Stars
 (41.258 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T 13h) (33 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Dieses Issue fordert, dass die Listenverkettung über mehrere Zeilen ähnlich wie Aufrufketten formatiert wird. Untersuche, wie Black derzeit binäre Ausdrücke mit List Comprehensions behandelt, insbesondere im blib2to3 oder lines Modul. Erwäge, die Split Logik so anzupassen, dass der + Operator am Anfang der Fortsetzungszeile platziert wird, entsprechend dem Stil, der für verkettete Aufrufe verwendet wird. Teste das bereitgestellte Beispiel, um sicherzustellen, dass die Ausgabe dem vorgeschlagenen Format entspricht.
Tech Stack
python
Domain
tooling
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
4
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Python 3.6+Familiarity with Black's codebase
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
35