search

QuiltMC / quilted-fabric-api

Re-implementation of the Fabric API using QSL as implementation instead.
Apache License 2.0
87 stars 32 forks source link

Migration to Quilt Testing API might have caused longer build times #76

Open EnnuiL opened 1 year ago

EnnuiL commented 1 year ago

I have a suspicion that this commit might have made development more painful by bringing longer build times and such. While I don't have concrete proof, I have tried reverting to QLoom 1.0 with that commit, and OOMs prevented that from working

I suspect that the reason this happens is due to the usage of modTestmodImplementation on QSL's modules and therefore having a need to remap all of them compared to Fabric simply pulling in it's module, which doesn't need remapping;

A solution would be reverting the migration while preventing the module from being published, which could work but wouldn't be ideal since we'll want to use Quilt's solution for game tests of our own;

Something must be investigated that keeps the usage of the Quilt Testing API while not causing the poor build system to suffer

EnnuiL commented 1 year ago

Update: I have a plan, I just need time