oakmac/standard-clojure-style-js

Remove blank behaviour in different cases

Open

#156 geöffnet am 10. Nov. 2024

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Clojure (7 Forks)github user discovery
bughelp wanted

Repository-Metriken

Stars
 (132 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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?

Contributor Guide

Research-Richtung
Untersuchen Sie die Handhabung von Leerzeilen durch den Formatierer in verschiedenen Editor Konfigurationen. Verfolgen Sie den Codepfad, der Leerzeichen und bedingte Gruppen verarbeitet. Bestimmen Sie, ob das Verhalten Leerzeichen respektieren oder konsistent ignorieren soll.
Tech Stack
javascriptclojure
Domain
developer experiencetooling
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
GitJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80