Harmonisierung von Zeilenumbrüchen

1DSAG / ABAP-Leitfaden

https://1dsag.github.io/ABAP-Leitfaden/

Creative Commons Attribution 4.0 International

7 stars 17 forks source link

Harmonisierung von Zeilenumbrüchen #42

Open fabianlupa opened 3 weeks ago

fabianlupa commented 3 weeks ago

Aktueller Stand:

Im git Repo sind bunt gemischt CRLF und LF enthalten.
Es werden unterschiedliches Tooling und unterschiedliche Betriebssysteme eingesetzt / Browserumgebung, welche die Einstellungen im Repo (editorconfig) unterschiedlich gut interpretieren.
Es wird angenommen, dass eine jetzige massenhafte Konvertierung auf LF viele "stört", weil es zu Merge Konflikten oder großen Diffs führen würde. Daher wird eine Konvertierung am Ende / zu einem Review bevorzugt.
Bis dahin sollte das Tooling so eingestellt werden, dass keine automatische Konvertierung der Zeilenenden erfolgt.

Status:

[ ] Automatische Konvertierung als Übergangslösung abschalten (#44)
[ ] Später: Konvertierung auf LF im Tooling wieder aktivieren
[ ] Später: Prüfung auf LF in CI integrieren
[ ] Später: Massenhafte Konvertierung aller Dateien auf LF
[ ] Später: LF dokumentieren

Siehe auch #36.

ABAP-Pete25 commented 2 weeks ago

ich hab im ADT Leitfaden mal als Zeilenumbruch
eingefügt. Richtiger scheint aber doppel Leerzeichen. Sollte im Status Meeting angesprochen werden da so langsam richtig Content entsteht.

fabianlupa commented 2 weeks ago

ich hab im ADT Leitfaden mal als Zeilenumbruch eingefügt. Richtiger scheint aber doppel Leerzeichen. Sollte im Status Meeting angesprochen werden da so langsam richtig Content entsteht.

Ich glaube du spielst auf den Markdown Syntax an mit doppelten Leerzeichen am Ende einer Zeile (überhaupt) einen Umbruch zu provozieren, obwohl kein Absatzwechsel entsteht (keine Leerzeile zwischen zwei Absätzen). Im Vergleich zur <br/> (was bei deinem Kommentar übrigens zu einem Umbruch wurde ;) ). Die Doppelleerzeichen werden in der Regel nur bei Aufzählungen oder ähnlichem benötigt. Das zu harmonisieren geht auch ganz am Ende. Die hier thematisierten Umbrüche haben Auswirkungen auf das Tooling und ggf. Build Setup, daher würde ich die Übergangslösung aus #44 auch gerne zeitnah mergen.

ABAP-Pete25 commented 2 weeks ago

OK Danke für die Hinweise und Hilfe. Gut dass wir drüber reden :-) . Ich gehe da nachher drüber und mach das "richtig" und stell dann einen gscheiten Pull Request. --- dieser Kommentar gehört in #48

fabianlupa commented 2 weeks ago

OK Danke für die Hinweise und Hilfe. Gut dass wir drüber reden :-) . Ich gehe da nachher drüber und mach das "richtig" und stell dann einen gscheiten Pull Request.

Passt :) Dafür ist das hier auch da.

Ich glaube dein Kommentar gehört in #48 statt hier.