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

#1255: Adding "Academic Title" field to persons in publish forms #1256

Merged
merged 2 commits into from
Jan 13, 2025

Conversation

alw-bsz
Copy link
Contributor

@alw-bsz alw-bsz commented Nov 19, 2024

No description provided.

@j3nsch j3nsch self-requested a review January 7, 2025 16:26
@j3nsch
Copy link
Member

j3nsch commented Jan 7, 2025

Wie kommt es zu den ganzen zusätzlich Änderungen in der Formatierung. Einzelne, entfernte Leerzeichen am Ende von Tags sind kein Problem, bzw. die Entfernung ist richtig. Die Entfernung von Leerzeilen zwischen Informationsblöcken ist an manchen Stellen sinnvoll, an anderen weniger schön. Diese Änderungen kann ich ignorieren, aber sie machen es natürlich schwer die eigentlich inhaltlichen Änderungen zu sehen.

Die Änderungen an den Headern, also das Entfernen der Leerzeichen am Anfang, ist nicht in Ordnung. Die Header sind absichtlich so formatiert gewesen, damit sie über alle Dateien hinweg identisch sind, zumindest der Hauptteil, der sich nicht von Datei zu Datei ändert. Das wird bitte rückgängig gemacht. Mir tut die zusätzliche Arbeit leid, aber Pull Requests sollten sich auf die eigentlichen inhaltlichen Änderungen beschränken.

In Tools like PhpStorm, IntelliJ IDEA und sicherlich auch anderen, lassen sich Änderungen an Dateien vor dem Commit einzeln prüfen und rückgängig machen. Ich weiß nicht warum bzw. wie die Header geändert wurden, aber damit hätte man diese Änderungen aus dem Commit heraushalten können.

@alw-bsz
Copy link
Contributor Author

alw-bsz commented Jan 10, 2025

Die Dateien hat ein*e Kolleg*in bearbeitet und zu intensiv Gebrauch von der "Pretty Print"-Funktionalität des Editors gemacht. Wir korrigieren die Formatierung.

@j3nsch
Copy link
Member

j3nsch commented Jan 13, 2025

Vielen Dank!

@j3nsch j3nsch merged commit 81ce7b9 into OPUS4:v4.8.1 Jan 13, 2025
1 check passed
@alw-bsz alw-bsz deleted the acadTitle1255 branch January 13, 2025 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants