QuiltMC / rfcs

Repository for requests for comments for proposing changes to the Quilt Project.
Other
61 stars 33 forks source link

Add a load type enum field #30

Closed AlexIIL closed 3 years ago

AlexIIL commented 3 years ago

Rich Diff

In order to support not loading modules if they aren't needed I'd like to add a new enum field load_type to quilt.mod.json. I should have opened this a while ago, sorry. (This is a port of FabricMC/fabric-loader#342)

AlexIIL commented 3 years ago

I marked this as a draft while I tested an implementation for this - which is done now.

i509VCB commented 3 years ago

My guess is plugin infrastructure would have to exist or rely on mods correctly declaring their dependencies to handle the loading?

AlexIIL commented 3 years ago

We could make plugins handle this, but I think it makes more sense to include this in the base mod defining? it's not very complicated functionality anyway.

TheGlitch76 commented 3 years ago

We missed the formal FCP but this PR is a minor change and has sat accepted for two weeks, so we're merging it now.