SteamWar/FightSystem
SteamWar/FightSystem
12
1
Fork 0
Code Issues 6 Pull-Requests Releases Wiki Aktivität

The big refactoring #238

Manuell gemergt
Lixfel hat 16 Commits von refactoring nach master 2021-03-31 20:08:07 +02:00 zusammengeführt
Diskussion 2 Commits 16 Geänderte Dateien 125 +4491 -3615
Lixfel hat 2021-03-10 17:12:54 +01:00 kommentiert
Besitzer
Link kopieren

Signed-off-by: Lixfel agga-games@gmx.de

Signed-off-by: Lixfel <agga-games@gmx.de>
Lixfel hat 1 Commit 2021-03-10 17:12:56 +01:00 hinzugefügt 
Signed-off-by: Lixfel <agga-games@gmx.de>
Lixfel hat den Titel von Some big refactoring zu WIP: Some big refactoring 2021-03-10 17:15:39 +01:00 geändert
Lixfel hat 1 Commit 2021-03-10 22:45:27 +01:00 hinzugefügt 
Signed-off-by: Lixfel <agga-games@gmx.de>
Lixfel hat 1 Commit 2021-03-12 15:43:35 +01:00 hinzugefügt 
Signed-off-by: Lixfel <agga-games@gmx.de>
Lixfel hat 1 Commit 2021-03-20 14:49:47 +01:00 hinzugefügt 
Signed-off-by: Lixfel <agga-games@gmx.de>
Lixfel hat 1 Commit 2021-03-20 17:25:18 +01:00 hinzugefügt 
Signed-off-by: Lixfel <agga-games@gmx.de>
Lixfel hat 1 Commit 2021-03-23 16:42:49 +01:00 hinzugefügt 
Signed-off-by: Lixfel <agga-games@gmx.de>
Lixfel hat 1 Commit 2021-03-23 19:54:08 +01:00 hinzugefügt 
Signed-off-by: Lixfel <agga-games@gmx.de>
Lixfel hat 1 Commit 2021-03-24 10:18:03 +01:00 hinzugefügt 
Signed-off-by: Lixfel <agga-games@gmx.de>
Lixfel hat 1 Commit 2021-03-24 11:21:20 +01:00 hinzugefügt 
Signed-off-by: Lixfel <agga-games@gmx.de>
Lixfel hat den Titel von WIP: Some big refactoring zu The big refactoring 2021-03-24 11:21:46 +01:00 geändert
Lixfel hat 1 Commit 2021-03-24 11:32:45 +01:00 hinzugefügt 
Signed-off-by: Lixfel <agga-games@gmx.de>
Lixfel hat ein Review von Chaoscaot 2021-03-24 11:33:20 +01:00 angefragt
Lixfel hat ein Review von YoyoNow 2021-03-24 11:33:20 +01:00 angefragt
Lixfel hat 2021-03-24 11:38:07 +01:00 kommentiert
Autor
Besitzer
Link kopieren

Alle Teile, welche in normalen MWG-Kämpfen benötigt werden, wurden bereits getestet.

Enthält an neuen Features:

  • Skip-Command zum nächsten Ereignis (countdown)
  • State-Command für den Test-Modus zum Wechseln des FightStates
  • Eine verkürzte & vereinfachte Konfiguration
  • Die Arena kann nach Ende des Kampfes/auch sonst wiederverwendet werden
  • Waterlogged Blöcke werden absofort mit gehided im Techhider
  • PacketPlayOutBlockBreak-Hider
  • Prepare-Arenenmodus, welcher es gestattet, die eigene Schem auszufahren.
  • Refactoring, was es einfacher machen sollte, folgende Features in Zukunft umzusetzen: Nutzung von Arenen für verschiedene Schemtypen, Kampf verschiedener Schemtypen gegeneinander, NonCentered/Aligned-Schempasting, 3D-Techhiderbypass
Alle Teile, welche in normalen MWG-Kämpfen benötigt werden, wurden bereits getestet. Enthält an neuen Features: * Skip-Command zum nächsten Ereignis (countdown) * State-Command für den Test-Modus zum Wechseln des FightStates * Eine verkürzte & vereinfachte Konfiguration * Die Arena kann nach Ende des Kampfes/auch sonst wiederverwendet werden * Waterlogged Blöcke werden absofort mit gehided im Techhider * PacketPlayOutBlockBreak-Hider * Prepare-Arenenmodus, welcher es gestattet, die eigene Schem auszufahren. * Refactoring, was es einfacher machen sollte, folgende Features in Zukunft umzusetzen: Nutzung von Arenen für verschiedene Schemtypen, Kampf verschiedener Schemtypen gegeneinander, NonCentered/Aligned-Schempasting, 3D-Techhiderbypass
Lixfel hat 1 Commit 2021-03-24 11:45:21 +01:00 hinzugefügt 
Signed-off-by: Lixfel <agga-games@gmx.de>
Lixfel hat 1 Commit 2021-03-24 13:49:24 +01:00 hinzugefügt 
Signed-off-by: Lixfel <agga-games@gmx.de>
YoyoNow hat 2021-03-26 08:22:44 +01:00 Änderungen angefragt
YoyoNow hat einen Kommentar hinterlassen
Besitzer
Link kopieren

Lixfel, sorry aber für die 2 Sachen die ich jetzt gefunden habe, habe ich ganze 20 Minuten gebraucht und die meiste Zeit davon ging damit drauf, dass ich auf meinen Browser gewartet habe, weil dieser diesen PR einfach nicht packt. Ich weiß echt nicht was du erwartest, was wir bei einem Peer Review hier machen sollen?

Lixfel, sorry aber für die 2 Sachen die ich jetzt gefunden habe, habe ich ganze 20 Minuten gebraucht und die meiste Zeit davon ging damit drauf, dass ich auf meinen Browser gewartet habe, weil dieser diesen PR einfach nicht packt. Ich weiß echt nicht was du erwartest, was wir bei einem Peer Review hier machen sollen?
FightSystem_Main/src/de/steamwar/fightsystem/states/StateDependent.java Veraltet
@ -1,45 +1,36 @@
/*
YoyoNow hat 2021-03-26 08:12:27 +01:00 kommentiert
Besitzer
Link kopieren

Licenz Fehlt

Licenz Fehlt
Lixfel markierte diese Unterhaltung als gelöst
FightSystem_Main/src/de/steamwar/fightsystem/states/StateDependentCommand.java Veraltet
@ -0,0 +28,4 @@
public class StateDependentCommand extends StateDependent {
private static final CommandExecutor unavailible = (sender, cmd, s, strings) -> {
YoyoNow hat 2021-03-26 08:11:03 +01:00 kommentiert
Besitzer
Link kopieren

unavailable

unavailable
Lixfel markierte diese Unterhaltung als gelöst
Lixfel hat 1 Commit 2021-03-27 17:57:54 +01:00 hinzugefügt 
Signed-off-by: Lixfel <agga-games@gmx.de>
Lixfel hat 1 Commit 2021-03-27 17:58:34 +01:00 hinzugefügt 
# Conflicts:
#	FightSystem_Main/src/de/steamwar/fightsystem/utils/TechHider.java
Lixfel hat 2021-03-27 18:14:09 +01:00 kommentiert
Autor
Besitzer
Link kopieren

YoyoNow, mir ist klar, dass eine klassische Überprüfung hier nicht so viel Sinn macht. Daher ist eher ein Betrachten der gundlegenden Struktur und Programmflussänderungen aus IntelliJ angedacht (das sind ja die meisten Zeilen) und ein Betrachten der Implementierung der neuen Features.

YoyoNow, mir ist klar, dass eine klassische Überprüfung hier nicht so viel Sinn macht. Daher ist eher ein Betrachten der gundlegenden Struktur und Programmflussänderungen aus IntelliJ angedacht (das sind ja die meisten Zeilen) und ein Betrachten der Implementierung der neuen Features.
Lixfel hat 1 Commit 2021-03-30 18:59:17 +02:00 hinzugefügt 
Signed-off-by: Lixfel <agga-games@gmx.de>
Lixfel hat 1 Commit 2021-03-30 19:38:59 +02:00 hinzugefügt 
Signed-off-by: Lixfel <agga-games@gmx.de>
Lixfel hat Commit 79164acbbf in master 2021-03-31 20:08:07 +02:00 manuell gemerged
Anmelden, um an der Diskussion teilzunehmen.
Reviewer
Keine Reviewer
Chaoscaot
YoyoNow
Label
Label entfernen
  
Bug

Etwas funktioniert nicht oder ist fehlerhaft.

  
Codeverbesserung

Ein Codeverbesserungsvorschlag

  
Einsteiger Freundlich

Ist für Leute welche noch nicht so eingearbeitet sind ein guter Start.

  
Idee

Bisher unausgereifte Idee für ein neues Feature

  
In Arbeit

Naja hier dran wird gearbeitet

  
Neues Feature

Das ist ein neues Feature

  
Prio A

Oberste Priorität

  
Security Breach

Die Sicherheit der User ist beeinträchtigt!

  
Überprüfung notwendig

Naja hier muss noch was überprüft werden?

  
Verbesserung

Ein Verbesserungsvorschlag

  
Zu Beobachten

Sollte gefixt sein bzw. kann bislang nicht bestätigt werden

Kein Label
Bug
Codeverbesserung
Einsteiger Freundlich
Idee
In Arbeit
Neues Feature
Prio A
Security Breach
Überprüfung notwendig
Verbesserung
Zu Beobachten
Meilenstein
Meilenstein entfernen
Keine Einträge
Kein Meilenstein
Zuständig
Zuständige entfernen
Niemand zuständig
3 Beteiligte
Nachrichten
Fällig am
Das Fälligkeitsdatum ist ungültig oder außerhalb des zulässigen Bereichs. Bitte verwende das Format „jjjj-mm-tt“.

Kein Fälligkeitsdatum gesetzt.

Abhängigkeiten

Keine Abhängigkeiten gesetzt.

Referenz: SteamWar/FightSystem#238
Keine Beschreibung angegeben.
Branch "refactoring" löschen

Das Löschen eines Branches ist permanent. Obwohl der Branch für eine kurze Zeit weiter existieren könnte, kann diese Aktion in den meisten Fällen NICHT rückgängig gemacht werden. Fortfahren?