Feature waffenwechsel

[pmayd]

refactoring of component into several helper functions

[pmayd]

Bitte für die Zukunft:

• Code halbwegs so formatieren wie meine Dateien formatiert sind...
• params immer ein Eintrag pro Zeile, auch wenn man kein default benutzt, damit man es später leicht hinzufügen kann!
• Mehrzeilige Arrays auf 1 Eintrag pro Zeile umschreiben, damit man es halbwegs lesen und maintainen kann (gilt z.B: für params oder classname-Arrays)
• Einrückungen! Jede Verschachtelung wird um 1 Ebene eingerückt! if-else, foreach, while usw. haben alle eine Einrückung um 4 Leerzeichen zur Folge. if-else sind auf einer Ebene. Klammern sind bitte immer in der selben Zeile nicht in einer extra
• if-then immer in einer eigenen Zeile, d.h. Code innerhalb immer in einer neuen Zeile! kein if (...) then { ... };
• LEERZEICHEN! Leerzeichen vor und nach einer Zuweisung "=", vor und nach Kommata (Ausnahme Makros), vor und nach mathematischen Operatoren (+,-,/,*,%) usw. Einheitlich und lesbar!
• kleines _x in forEach, kein großes!
• deine Funktionen haben keinen vollständigen Header, es fehlt die Beschreibung, was das Skript tut! Beispiel bewaffnen.sqf
• kein private wenn die Variable mit params bereits definiert wurde, params legt jede Variable als private an!

WICHTIG: Im Skript bewaffnen.sqf taucht eine Variable wert auf, die nicht übergeben wird. Woher kommt die?! So kann es nicht gehen

[senshi-x]

Ui, ein echter, richtiger Pull Request erledigt 🎉 . Gratulation!

[pmayd]

We kommt es eigentlich zum Close des FR? Macht er das automatisch, weil der branch in einen anderen gemergt wurde?

Tatsächlich, der Status ist ja auch merged....schon toll

[senshi-x]

Jep, ein PR wird automatisch geschlossen, wenn er gemergt wird. Weil mergen tut man ja erst, wenn das eigentliche Thema erledigt ist, mit all seinen Änderungswünschen etc. .

EDIT: Zur Klärung: Der PR ist ja immer nur ein "ich will branch X auf branch Y mergen". Das hängt solange in der Schwebe, bis man es akzeptiert. Wenn einem noch was fehlt, wird branch X solange aktualisiert, bis man zufrieden ist. Dann mergt man X auf Y, und damit hat sich der PR erledigt.