VazkiiMods / Botania

A tech mod for Minecraft themed around the magic of nature and plant life.
http://botaniamod.net/index.html
Other
1.27k stars 505 forks source link

Separation of AncientWillContainer and TerrasteelHelmItem #4749

Open megadoxs opened 4 weeks ago

megadoxs commented 4 weeks ago

These changes should allow for a better implementation by other mods of the AncientWillContainer interface to new armor pieces. Works for both fabric and forge.

Should terraWillCritTarget inside PlayerMixin be refactored to WillCritTarget as it will not contain only a target added by the terrahelm, but by all items that implements AncientWillContainer?

Fixes https://github.com/VazkiiMods/Botania/issues/4746

PS: Sorry for closing and remaking this pr, had messed up my fork.