Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Harmonisierung von Zeilenumbrüchen #42

Open
5 tasks
fabianlupa opened this issue Oct 30, 2024 · 4 comments
Open
5 tasks

Harmonisierung von Zeilenumbrüchen #42

fabianlupa opened this issue Oct 30, 2024 · 4 comments

Comments

@fabianlupa
Copy link
Collaborator

fabianlupa commented Oct 30, 2024

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 (Deaktivierung automatischer Zeilenumbruchkonvertierung #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
Copy link
Contributor

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
Copy link
Collaborator Author

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
Copy link
Contributor

ABAP-Pete25 commented Nov 2, 2024

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
Copy link
Collaborator Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants