search

letsgamedev / Suffragium

A game developed in a democratic cycle.
GNU Affero General Public License v3.0
51 stars 21 forks source link

Create a simple workflow to lint the code #16

Closed Joshix-1 closed 2 years ago

Joshix-1 commented 2 years ago

Das ist ein einfacher Workflow, der automatisch alle GD-Skript-Dateien lintet.

Dieser PR basiert auf dem Vorschlag in #10

RedstoneMedia commented 2 years ago

Also ich finds prinzipiell ganz gut aber ich würde ne paar von den Default Regeln rausnehmen.

Joshix-1 commented 2 years ago

Welche denn? Ich find da keine Regel doof

RedstoneMedia commented 2 years ago

Ich find besonders die erzwungen Code Reihenfolge nervig. Die Kommas am ende von Enums/Listen find ich auch immer unschön. Die anderen Regeln finde ich sonst in Ordnung.

Joshix-1 commented 2 years ago

Eine erzwungene Code-Reihenfolge macht Dinge übersichtlicher.

Und das Komma am Ende von Enums/Listen find ich gut. Das macht Diffs leserlicher, wenn man am Ende neue EInträge hinzufügt oder Sachen umsortiert.

RedstoneMedia commented 2 years ago

Ja, das mit den Kommas find ich noch ganz ertragbar, aber die Code Reihenfolge, die Die vorschlagen passt mit meinem Codestyle oft nicht zusammen. Klar class_name und tool kommt zuerst und dann die globalen Variablen, aber alles was da drüber hinaus geht finde ich einfach nur lästig und hilft mir persönlich nicht den Code besser/schneller zu verstehen.

ASecondGuy commented 2 years ago

Mich stört dass der die tabs weg macht wenn kein code in der Zeile steht. Das iritiert wenn mitten in einer funktion die Tabs fehlen. Edit: Das erste ist nicht so übersichtlich wie das 2. grafik grafik