SkriptLang / skript-aliases

Aliases for Skript
19 stars 35 forks source link

[Suggestion] Removal of Items w/ ArmorTrim aliases #111

Open ShaneBeee opened 3 months ago

ShaneBeee commented 3 months ago

I would like to suggest the removal of the aliases for items with armor trims. To start:

[Skript] Loaded 232659 aliases in 4296ms # default
[Skript] Loaded 197964 aliases in 2638ms # without armor trim
sovdeeth commented 3 months ago

I'd be in favor of this.

Moderocky commented 3 months ago

Absolutely agree, we're planning to start deprecating aliases as of 2.9 anyway and moving towards and optional (opt-in) model in future.

Fusezion commented 3 months ago

Could be made into the same system used for potions +1

Moderocky commented 3 months ago

moving towards and optional (opt-in) model in future.

Our preliminary idea is to cut these out and provide an armor-trims.sk that people can drop in to restore the current usage. The same can probably be done for other (problematic) alias groups, before the complete removal.