Elias Müller MineTec
0 Followers · 0 Following
Repositories
2
 Projects Packages Public Activity Starred Repositories
2
MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:24:06 +00:00
develop-chatReply

this.sendWhisper

MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:21:22 +00:00
develop-chatReply

initialisierung und sofortig bedingtes überschreiben würde ich als bad practice einstufen...

MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:16:27 +00:00
develop-chatReply

die null checks überall sind nicht schön, es macht vielleicht eher sinn am anfang einmal zu prüfen ob ein Eintrag vorhanden ist und wenn nicht ne leere Liste zu initialisieren..

MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:15:08 +00:00
develop-chatReply

hier wird ganz oft this.replyMapping.get(sender) verwendet, rausziehen in eine eigene variable z.B. replyMap = this.replyMapping.get(sender)

MineTec pushed to master at CraftAttack/spawn 2024-10-05 16:40:54 +00:00
346847d2b2 Merge pull request 'master-customAdvancements' (#4) from master-customAdvancements into master
e3b07aa62f solved pr comments
da33e6e592 added more custom advancements, added pending advancements
1aebae6cd5 added fleischerchest and pixelblock advancements
Compare 4 commits »
MineTec deleted branch master-customAdvancements from CraftAttack/spawn 2024-10-05 16:40:54 +00:00
MineTec merged pull request CraftAttack/spawn#4 2024-10-05 16:40:52 +00:00
master-customAdvancements
MineTec approved CraftAttack/spawn#4 2024-10-05 16:40:39 +00:00
master-customAdvancements
MineTec opened issue CraftAttack/website#31 2024-10-03 19:56:28 +00:00
Anmeldebutton auf startseite bei geschlossener Anmeldung austauschen
MineTec commented on pull request CraftAttack/spawn#4 2024-09-29 17:05:45 +00:00
master-customAdvancements

mein Vorschlag hier wäre du throwst nach dem custom fehler einfach nochmal die exception throw e und packst in die Objects.requireNonNull das angefragte rein. Dann ist alles immer ersichtlich.

MineTec suggested changes for CraftAttack/spawn#4 2024-09-29 17:03:36 +00:00
master-customAdvancements

sieht gut aus :)

MineTec commented on pull request CraftAttack/spawn#4 2024-09-29 16:57:35 +00:00
master-customAdvancements

das else if konstrukt ist nicht schön...

MineTec commented on pull request CraftAttack/spawn#4 2024-09-29 16:56:36 +00:00
master-customAdvancements

der check nach enchantmentGlintOverride raus, itemName reicht aus

MineTec commented on pull request CraftAttack/spawn#4 2024-09-29 16:53:08 +00:00
master-customAdvancements

sollte private sein, wenn grantAdvancement entscheidet ob es pending ist oder nicht

MineTec commented on pull request CraftAttack/spawn#4 2024-09-29 16:52:19 +00:00
master-customAdvancements

die Fehlermeldung kann irreführend sein. Es sollte zumindest im Fehler ersichtlich sein welches advancement fehlt.

MineTec commented on pull request CraftAttack/spawn#4 2024-09-29 16:49:45 +00:00
master-customAdvancements

?

MineTec pushed to develop-chatReply at CraftAttack/spawn 2024-09-29 14:40:51 +00:00
9390d8c67b Merge remote-tracking branch 'refs/remotes/origin/master' into develop-chatReply
bf94e152c8 changed snowball knockback
3c1dea3451 Merge remote-tracking branch 'origin/master'
36fc89e915 added snowball knockback
7a15984f19 added snowball knockback
Compare 28 commits »
MineTec pushed to master at CraftAttack/spawn 2024-09-27 16:16:46 +00:00
70a1740644 fixed rainbowComponent not folding back on end of color spectrum, ensuring no sudden color switch
MineTec pushed to master at CraftAttack/spawn 2024-09-27 16:02:29 +00:00
24fbd50c62 added appendWithSpace component util
MineTec pushed to master at CraftAttack/spawn 2024-09-27 15:09:48 +00:00
05e88845be network monitor now disables itself when not being able to read system stats properly