psf/black

List concatenation, multiple lines

Open

#260 aberto em 26 de mai. de 2018

Ver no GitHub
 (2 comments) (3 reactions) (0 assignees)Python (2.693 forks)batch import
F: linebreakF: symmetryT: enhancementT: stylehelp wanted

Métricas do repositório

Stars
 (41.258 stars)
Métricas de merge de PR
 (Mesclagem média 8d 13h) (33 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Esta issue solicita que a concatenação de listas em múltiplas linhas seja formatada de forma semelhante a cadeias de chamadas. Investigue como o Black atualmente lida com expressões binárias envolvendo list comprehensions, particularmente no módulo blib2to3 ou lines. Considere ajustar a lógica de divisão para que o operador + seja colocado no início da linha de continuação, alinhando se com o estilo usado para chamadas encadeadas. Teste o exemplo fornecido para garantir que a saída corresponda à formatação proposta.
Pilha de tecnologia
python
Domain
tooling
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
4
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
Python 3.6+Familiarity with Black's codebase
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
35