1DSAG / ABAP-Leitfaden

https://1dsag.github.io/ABAP-Leitfaden/
Creative Commons Attribution 4.0 International
7 stars 17 forks source link

chore: add authors links #34

Closed marianfoo closed 1 month ago

marianfoo commented 1 month ago

Ich denke es ist sinnvoll bei jedem Namen ein Link nach Wahl zu hinterlegen (webseite, LinkedIn, Mastodon etc.). Was denkst du @Xexer ? Das Diff schaut schlimmer aus als es ist, habe nur einen Link zu deinem Namen hinzugefügt.

image

fabianlupa commented 1 month ago

Das Diff schaut schlimmer aus als es ist, habe nur einen Link zu deinem Namen hinzugefügt.

Haben wir hier ein Zeilenenden-Problem?

marianfoo commented 1 month ago

Das Diff schaut schlimmer aus als es ist, habe nur einen Link zu deinem Namen hinzugefügt.

Haben wir hier ein Zeilenenden-Problem?

Auf den ersten Blick ja, aber lokal ist der diff korrekt, konnte es aber nicht beheben

fabianlupa commented 1 month ago

Hmm, sieht mir so aus, als wäre die in git gespeicherte Version inklusive CRLF. Best practice wäre sicherlich in git nur mit LF zu arbeiten und alle Windows User haben core.autocrlf=true an (oder ähnliches). Das sollten wir einmal massenhaft migrieren und eine Prüfung einbauen.

marianfoo commented 1 month ago

Du kannst gerne einen eigenen PR machen und rebase dann die Branch hier, ich wüsste nicht wie ich vorgehen müsste

Xexer commented 1 month ago

@marianfoo Genau, Links waren geplant, habe erst einmal nur die Liste übernommen. Danach müssen Firmen und Rollen noch gefüllt werden.

fabianlupa commented 1 month ago

Sieht in der Tat bunt gemischt aus. Ich schaue mal, ob ich eine dos2unix PR aufgemacht bekomme. Das könnte allerdings ziemlich stören, wenn parallel Personen in ihren Branches arbeiten.

fabian@ubuntu:~/Code/ABAP-Leitfaden$ find ./ -name '*.md' -or -name '*.html' | xargs file
./docs/core-data-services/index.md:                          ASCII text, with CRLF line terminators
./docs/_includes/wip-disclaimer.md:                          Unicode text, UTF-8 text
./docs/_includes/footer_custom.html:                         HTML document, Unicode text, UTF-8 text, with very long lines (451)
./docs/_includes/toc_landing_page.html:                      HTML document, ASCII text
./docs/_includes/search_placeholder_custom.html:             ASCII text, with no line terminators
./docs/_includes/toc_heading_custom.html:                    ASCII text, with no line terminators
./docs/abap/img/index.md:                                    ASCII text, with CRLF line terminators
./docs/user-interface/index.md:                              ASCII text, with CRLF line terminators
./docs/security/index.md:                                    ASCII text, with CRLF line terminators
./docs/application-lifecycle-management/success-factors.md:  Unicode text, UTF-8 text
./docs/application-lifecycle-management/ensuring-quality.md: Unicode text, UTF-8 text
./docs/application-lifecycle-management/index.md:            Unicode text, UTF-8 text, with CRLF line terminators
./docs/integration/index.md:                                 ASCII text, with CRLF line terminators
./docs/forms/index.md:                                       ASCII text, with CRLF line terminators
./docs/_layouts/home.html:                                   ASCII text
./docs/_layouts/page.html:                                   ASCII text
./docs/artificial-intelligence/index.md:                     Unicode text, UTF-8 text, with very long lines (789), with CRLF line terminators
./docs/introduction/index.md:                                Unicode text, UTF-8 text, with very long lines (760)
./docs/organization/index.md:                                ASCII text, with CRLF line terminators
./docs/authors/index.md:                                     HTML document, Unicode text, UTF-8 text, with CRLF line terminators
./docs/clean-core/what-is-clean-core.md:                     Unicode text, UTF-8 text, with very long lines (543), with CRLF line terminators
./docs/clean-core/why-clean-core.md:                         Unicode text, UTF-8 text, with CRLF line terminators
./docs/clean-core/solution-approach.md:                      Unicode text, UTF-8 text, with very long lines (654), with CRLF line terminators
./docs/clean-core/index.md:                                  Unicode text, UTF-8 text, with CRLF line terminators
./docs/clean-core/architectural-concepts.md:                 Unicode text, UTF-8 text, with CRLF line terminators
./docs/clean-core/problems-and-challenges.md:                Unicode text, UTF-8 text, with very long lines (572), with CRLF line terminators
./docs/documentation/index.md:                               ASCII text, with CRLF line terminators
./docs/version-management/index.md:                          ASCII text, with CRLF line terminators
./docs/404.html:                                             HTML document, ASCII text
./docs/open-source/developing-open-source.md:                Unicode text, UTF-8 text
./docs/open-source/open-source-projects.md:                  Unicode text, UTF-8 text
./docs/open-source/abapgit-as-enabler.md:                    Unicode text, UTF-8 text
./docs/open-source/index.md:                                 Unicode text, UTF-8 text, with very long lines (744), with CRLF line terminators
./docs/open-source/using-open-source.md:                     Unicode text, UTF-8 text
./docs/open-source/contributing-to-open-source.md:           Unicode text, UTF-8 text
./docs/index.md:                                             Unicode text, UTF-8 text
./docs/testing/other_test_tools.md:                          Unicode text, UTF-8 text, with CRLF line terminators
./docs/testing/abap_unit.md:                                 Unicode text, UTF-8 text, with very long lines (344), with CRLF line terminators
./docs/testing/index.md:                                     Unicode text, UTF-8 text, with CRLF line terminators
./README.md:                                                 Unicode text, UTF-8 text, with very long lines (390)
./contributing.md:                                           Unicode text, UTF-8 text
marianfoo commented 1 month ago

da @Xexer sowieso geplant hat die links noch zu ergänzen schließe ich den PR mal wieder und @fabianlupa kann einen neuen aufmachen für die line ending probleme