search

QuiltMC / quilt-loom

MIT License
22 stars 17 forks source link

Add Quilt Mappings support #4

Closed IotaBread closed 2 years ago

IotaBread commented 3 years ago

Update to Loom 0.9, update Quiltflower to 1.5.0, change task group to Quilt, un-hardcode some constants. Tests still need updating.

i509VCB commented 3 years ago

I think it would be more appropriate to have 0.9 on a branch for 0.9 instead of merging it into 0.8.

Also doesn't 0.9 add a requirement to have Java 16 if I recall?

IotaBread commented 3 years ago

I agree. I merged FabricMC/dev/0.9 into QuiltMC/dev/0.8 because the latter had some work done on it. I have a branch without the QM support that could be merged into a new dev/0.9, which this pr would then target.

And yes, loom 0.9 does require Java 16, but still can be used for Java 8 mods by setting sourceCompatibility and targetCompatibility in a mod's buildscript to Java 8. I still can open a PR with QM support for 0.8 if you want

Siuolplex commented 3 years ago

This is really useful for me since Im making some early quilt mods. When I get home I'll test and see if it works. Thank you!

TheGlitch76 commented 2 years ago

I'm doing a fresh restart of this loom fork, sorry! Please cherry-pick this back on to dev/0.10