@tgrothe Danke für den Beitrag. Drei Beobachtungen:

1. Im ersten Kommentar des PR bitte immer der Dreiklang:
   • Was ist kaputt? (Was habe ich beobachtet, was habe ich erwartet?)
   • Was ist der Fix? (Was ist der Lösungsansatz?)
   • Gibt es mögliche API- oder Verhaltensänderungen, bricht für irgendwen irgendwas?
2. Warum fährst Du für eine simple Umbenennung die ganz großen Geschütze auf und forderst ein Review von gleich mehreren Leuten, zumal @AMatutat im Moment gar nicht aktiv im Projekt arbeitet? Das führt nur dazu, dass er Dich/uns auf "ignore" setzt und dann in wirklich wichtigen Fällen nicht mehr reagiert, weil er es nicht mehr mitkriegt :/
3. Was ist der Sinn hinter den "komplizierten" Optionen? Reicht nicht ein simples grep -r "deamon"? Mit dem -C 1 clutterst Du die Ausgabe mit lauter hier nicht relevanten Angaben?

@tgrothe Ich stelle diesen PR noch einmal zurück. Wir haben zwei abhängige Projekte (Forks), die in den nächsten Wochen gemergt werden sollen. Bei euch im Dojo könntet ihr die Änderung mit rüberziehen, dann wäre der Merge kein Thema. Beim Dev ist die aktive Entwicklung vorbei afaik, d.h. das gibt dann nach dem Merge Probleme, weil dann die Textur fehlt und man sich erst an diese Umbenennung hier erinnern muss.

Note2Self: Im Dev-Dungeon ist die Klasse devDungeon/src/entities/MonsterType.java: "character/monster/big_deamon" betroffen.

Moin und Danke erst mal. ☕

2. Warum fährst Du für eine simple Umbenennung die ganz großen Geschütze auf und forderst ein Review von gleich mehreren Leuten, zumal @AMatutat im Moment gar nicht aktiv im Projekt arbeitet? Das führt nur dazu, dass er Dich/uns auf "ignore" setzt und dann in wirklich wichtigen Fällen nicht mehr reagiert, weil er es nicht mehr mitkriegt :/

War ein Versehen, bzw. wurde als Erstes von GitHub vorgeschlagen. Nun weiß ich Bescheid.

3. Was ist der Sinn hinter den "komplizierten" Optionen? Reicht nicht ein simples grep -r "deamon"? Mit dem -C 1 clutterst Du die Ausgabe mit lauter hier nicht relevanten Angaben?

Wenn es in den Docs dazu etwas gegeben hätte, wäre -C 1 wegen des Kontexts sinnvoll gewesen ... das gab es aber nicht. Mea culpa.

@tgrothe Mir ist aufgefallen, dass es noch einen weiteren Typo bei den Dämonen gibt :) ... Die Texturen selbst werden nur "demon" statt "daemon" geschrieben: dungeon/assets/character/monster/big_daemon/idle/big_demon_idle_anim_f0.png ... Das sollte mit angepasst werden.

Hintergrund: Demon und Daemon haben eine unterschiedliche Bedeutung - ein Demon ist eine negative Entität, während ein Daemon eine hilfreiche Entität mit supernatürlichen Kräften ist. Wir wollen die letztere Bedeutung, auch in Anlehnung an die Unix-Daemons :)

Danke für das Recherchieren. :) Ich nahm an, demon wäre nur eine andere Schreibweise für daemon, und deamon wäre in jedem Fall falsch ...

Aber dann weiß ich nun Bescheid und kann "später" den "demon" (bzw. die demons ...) auch noch umbenennen.

Sieht gut aus:

$ grep -r "deamon" --exclude-dir=.git --exclude-dir=.gradle --exclude-dir=.idea || grep -r "demon" --exclude-dir=.git --exclude-dir=.gradle --exclude-dir=.idea || find . -name '*deamon*' -not -path './.git/*' -exec bash -c 'echo mv $0 ${0/deamon/daemon}' {} \; || find . -name '*demon*' -not -path './.git/*' -exec bash -c 'echo mv $0 ${0/demon/daemon}' {} \;

dungeon/src/contrib/components/ItemComponent.java: * <p>The {@link contrib.entities.WorldItemBuilder WorldItemBuilder} demonstrates how to create an

"demonstrates" können wir ignorieren

sieht gut aus, danke - ich stelle das trotzdem nochmal zurück, #1485 (comment) gilt immer noch :)

jup, danke ... heute morgen hatte ich auch noch nicht genug kaffee gehabt. ;)