Terasology / SimpleFarming

Easy to understand growing of foods
11 stars 27 forks source link

feat: split GenomeAuthoritySystem based on modules #120

Closed skaldarnar closed 3 years ago

skaldarnar commented 3 years ago

The integration with Genome and BasicCrafting is optional for SimpleFarming.

As a component system can only be loaded properly if all it's dependencies are also active, this means that the combined GenomeAuthoritySystem is only loaded if both Genome and BasicCrafting are active.

By splitting the authority system into GenomeExtensionAuthoritySystem and GenomeCraftingExtensionAuthoritySystem each extension system can run with the minimal set of required modules.

Fixes #119 Closes #103

keturn commented 3 years ago

Does this also fix #118?

skaldarnar commented 3 years ago

Doesn't look like the tests are fixed: see pipeline