oakmac/standard-clojure-style-js

Remove blank behaviour in different cases

Open

#156 aberto em 10 de nov. de 2024

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)Clojure (7 forks)github user discovery
bughelp wanted

Métricas do repositório

Stars
 (132 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Hi, just watched your conj talk, this is cool - your efforts are appreciated.

Came across two different cases in different editors Pic instead code block because it shows blankspaces.

In first case, which by default in my vscode setup will be formatted to

(cond
  :cond-1
  :do-1
  :cond-2
  :do-2
  :cond-3
  :do-)

but(which default in my cursive setup) will be formatted with respect to blank lines to

(cond
  :cond-1
  :do-1

  :cond-2
  :do-2

  :cond-3
  :do-)

I would personally prefer to respect empty lines, but I wonder which behavior is considered preferable and whether the formatter behaves correctly in both cases?

Guia do colaborador

Direção de pesquisa
Investigue como o formatador lida com linhas em branco em diferentes configurações de editor. Rastreie o caminho do código que processa espaços em branco e grupos condicionais. Determine se o comportamento deve respeitar linhas em branco ou ignorá las de forma consistente.
Pilha de tecnologia
javascriptclojure
Domain
developer experiencetooling
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
GitJavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80