-
-
Notifications
You must be signed in to change notification settings - Fork 1
Make Controlling Mixin only load if Controlling is installed #19
Copy link
Copy link
Closed
Labels
BugSomething isn't working as expectedSomething isn't working as expectedFabricIssues occurring on the Fabric platformIssues occurring on the Fabric platformForgeIssues occurring on the Forge platformIssues occurring on the Forge platformMinecraft 1.21.11Issues occurring in Minecraft 1.21.11Issues occurring in Minecraft 1.21.11NeoForgeIssues occurring on the NeoForge platformIssues occurring on the NeoForge platform
Metadata
Metadata
Assignees
Labels
BugSomething isn't working as expectedSomething isn't working as expectedFabricIssues occurring on the Fabric platformIssues occurring on the Fabric platformForgeIssues occurring on the Forge platformIssues occurring on the Forge platformMinecraft 1.21.11Issues occurring in Minecraft 1.21.11Issues occurring in Minecraft 1.21.11NeoForgeIssues occurring on the NeoForge platformIssues occurring on the NeoForge platform
Type
Projects
Status
🎉 Done
Minecraft Version
1.21.11
Mod Loader
NeoForge / Fabric / Forge
Describe the Issue
Right now if Controlling is not installed it leads to this error message in the log:
It's harmless, but may lead users to make reports or incorrectly attribute errors to Kuma/Balm.
Initial attempt at adding a Mixin Plugin to make it conditional lead to weird class mismatches (possibly due to different Mixin versions/flavors in the subprojects?)... we'll have to see.
Logs
dev