Clarify commands defined in kits
YanisBft opened this issue · 2 comments
Feature description
Even though kits can contain commands executed when you receive a kit, it's not really clear to have them defined in items
list.
I suggest to move commands to a commands
list, as shown in the simple example below.
Before:
kits:
dtools:
delay: 600
items:
- dpickaxe 1 efficiency:1 durability:1 fortune:1 name:&4Gigadrill lore:The_drill_that_&npierces|the_heavens
- dshovel 1 digspeed:3 name:Dwarf lore:Diggy|Diggy|Hole
- lhelm 1 color:255,255,255 name:Top_Hat lore:Good_day,_Good_day
- daxe:780 1
- /broadcast {USERNAME} just got some fancy tools!
After:
kits:
dtools:
delay: 600
items:
- dpickaxe 1 efficiency:1 durability:1 fortune:1 name:&4Gigadrill lore:The_drill_that_&npierces|the_heavens
- dshovel 1 digspeed:3 name:Dwarf lore:Diggy|Diggy|Hole
- lhelm 1 color:255,255,255 name:Top_Hat lore:Good_day,_Good_day
- daxe:780 1
commands:
- broadcast {USERNAME} just got some fancy tools!
How the feature is useful
Helps to clarify that you can actually define commands, and avoid to rely on a single slash.
The comments at the top of that file are pretty clear about how to add it, but I’ll wait for comments from a higher power before changing status of this.
I'm curious exactly what does this really solve? I don't think it's particularly hard to tell which ones are commands and which are items in the list - commands have a /
before them, while items don't.
The other thing is that right now you can be certain that items and commands will be given/run in the order given in the config, which is useful for eg crates plugins that use commands to give special items with custom NBT, but that guarantee falls apart if you split items and commands out. I'd rather not break this behaviour unless it's for a good reason.