Update patchwork-events-rendering

PatchworkMC / patchwork-api

An attempt to reimplement the Minecraft Forge API on Fabric

GNU Lesser General Public License v2.1

283 stars 48 forks source link

Update patchwork-events-rendering #221

Closed stairman06 closed 3 years ago

stairman06 commented 3 years ago

Ports patchwork-events-rendering to 1.16. This PR also updates the Fabric API requirement to 0.29.0 as it uses the new WorldRenderEvents.

This also brings back ModLoader in patchwork-fml, so I can call postEvent, I don't think that should break anything though. The actual implementation of postEvent here doesn't actually match the forge implementation (see here), because I thought implementing IModBusEvent would be too out of scope for this PR.

stairman06 commented 3 years ago

Don't merge this yet! I've encountered a strange error when doing some testing and I want to verify that it's not this PR causing the problem.

stairman06 commented 3 years ago

After some local testing I'm unable to fix the issue. Either way, I think the issue being because of this PR is unlikely - all the events fire at the right spot. There is probably something else I'm missing, or Forge is patching something else that I've missed. Anyway, I think this PR is good for now, it implements all the previous events.