SteamWar/BungeeCore
Archiviert
13
2

Fix some SonarLint Issues and Modernize Java #517

Zusammengeführt
Lixfel hat 3 Commits von modernize nach velocity 2024-06-20 22:20:50 +02:00 zusammengeführt
Nur Änderungen aus Commit 101a8ed1e4 werden angezeigt - Alle Commits anzeigen

Datei anzeigen

@ -31,7 +31,6 @@ import lombok.RequiredArgsConstructor;
import net.kyori.adventure.text.Component;
import net.kyori.adventure.text.serializer.legacy.LegacyComponentSerializer;
import net.kyori.adventure.title.Title;
import org.jetbrains.annotations.NotNull;
Lixfel markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Sr, was mir hier gerade wieder eingefallen ist: NotNull und Nullable sind ganz böse (müssen weg). In IntelliJ hast du das Jetbrains-Package, der Buildserver eher weniger... Deswegen bitte rausentfernen.

Sr, was mir hier gerade wieder eingefallen ist: NotNull und Nullable sind ganz böse (müssen weg). In IntelliJ hast du das Jetbrains-Package, der Buildserver eher weniger... Deswegen bitte rausentfernen.
Veraltet
Review

War hier die einzige Stelle.

War hier die einzige Stelle.
Veraltet
Review

Ist durch JDA auch im Build-Path, sollte also auch in der CI-Bauen, kanns aber entfernen.

Ist durch JDA auch im Build-Path, sollte also auch in der CI-Bauen, kanns aber entfernen.
import java.time.Duration;
import java.util.*;
@ -190,7 +189,7 @@ public class EloPlayerHandler extends PacketHandler {
}
}
private static @NotNull IntFunction<String> getEmblemTransition(String oldEmblem, String newEmblem, String arrowColor) {
private static IntFunction<String> getEmblemTransition(String oldEmblem, String newEmblem, String arrowColor) {
String finalOldEmblem = (oldEmblem.isEmpty() ? "/" : oldEmblem).trim();
String finalNewEmblem = (newEmblem.isEmpty() ? "/" : newEmblem).trim();