Ja, der Titel sagt eigentlich alles.
Ich erwisch mich immer wieder, gerade durch andere Frameworks auch voran getrieben, dass ich im WCF Classname.php schreibe, statt Classname.class.php. Wäre schön, wenn man das Verhalten des WCFs etwas an das von anderen Frameworks anpasst.
fände ich auch interessant, vor Allem weil überlange dateinamen vielleicht auf servern die ohnehin schon recht lange base-pfade haben (kann ggf bei sharedservern vorkommen) dass es nicht zu "zu langer Pfad" kommt.
abhängig vom OS sind ja gerne nur 2 hundert sonstwas zeichen erlaubt.
Ich war mal so frei einen PR dafür zu erstellen:
https://github.com/WoltLab/WCF/pull/2036
classPath_withoutClass ist eine Mischung aus Camel- und KebapCase 
Und laut Vorgabe sollte meines Wissens nach das else { eine eigene Zeile bekommen. 
Danke euch beiden, habe es geändert und den PR aktualisiert.
Wäre fast schöner, du würdest den PR neu erstellen. Weil 4 Commits für 5 Zeilen... Ich weiß ja nicht
Hallo,
Wäre fast schöner, du würdest den PR neu erstellen. Weil 4 Commits für 5 Zeilen... Ich weiß ja nicht
mittlerweile könnte man bei GitHub direkt die Commits in einen einzelnen Commits squashen, das ist also das kleinste Problem. Sonst habe ich es teilweise auch schon selbst gesquasht.
Ah, okay
Hallo,
mittlerweile könnte man bei GitHub direkt die Commits in einen einzelnen Commits squashen, das ist also das kleinste Problem. Sonst habe ich es teilweise auch schon selbst gesquasht.
Kurzes Off-Topic: Kann das nur die Person die Mergen kann oder kann man es auch "selber"?
Hallo,
Kurzes Off-Topic: Kann das nur die Person die Mergen kann oder kann man es auch "selber"?
das kannst du grundsätzlich auch selbst: git rebase -i master (master ist durch deinen Basisbranch zu ersetzen) und dann bei den nachfolgenden Commits fixup statt pick wählen. Anschließend einen Force-Push durchführen, nachdem du mit git diff HEAD^..HEAD geschaut hast, ob das Ergebnis vernünftig aussieht.
Ich meinte eigentlich in Github 
aber danke.
Soll ich den PR denn nun squashen oder machst du es sofern du (einer von euch) ihn merged?
Hallo,
Das squashen machen wir.
Don’t have an account yet? Register yourself now and be a part of our community!
