search

letsgamedev / Suffragium

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

adding player controller #7

Closed schluggar closed 2 years ago

schluggar commented 2 years ago

Ist mein erster Pull-Request. Dementsprechend alles ein wenig holperig. Kann ich solange der Request noch offen ist commits dafür durchführen? Ich hab nämlich gesehen, dass die .md-Datei wieder zurückgesetzt wurde. (Frage hat sich geklärt :))

VatrasCell commented 2 years ago

Mhh, ich weiß nicht, ob wir jetzt schon einen Player Controller brauchen, wo wir noch gar nicht wissen bzw. geklärt haben, was für ein Spiel das hier werden soll. Ich würde das hier also erstmal vertagen bis das ansatzweise geklärt ist. Dann kann man das ja immer noch mergen, wenn es gebaucht wird.

schluggar commented 2 years ago

Dann würde den Request nochmal rausnehmen und dann nochmal einstellen, falls es benötigt wird.

ckanitz commented 2 years ago

Dann würde den Request nochmal rausnehmen und dann nochmal einstellen, falls es benötigt wird.

Du solltest den PR auch auf "Draft" stellen können. Das sollte für's erste reichen. Dann sehen auch andere, dass da schonmal was getan wurde.

dzatoah commented 2 years ago

Also irgendwas hast du auch grob falsch giemacht (nicht böse gemeint :D), weil der README commit plötzlich von dir commited ist und im commit danach löschst du den Inhalt der README wieder. (git pull vergessen?) Sonst einmal alle Änderungen wegsichern und nochmal commiten

oder gegen main rebasen. git rebase -i master vorrausgesetzt du hast deinen branch schluggar:adding-2d-player-controller ausgecheckt

joshua-Ig commented 2 years ago

Ich finde n Player Controller nice wenigstens hat man dan irgend was um drauf aufzubauen