-
-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MixinExtras is set up incorrectly #83
Comments
Ah, wasn't made aware that it needs to be relocated. Will look into it next week(I'm off to a festival, so not at home). |
The whole setup process is detailed in the readme. |
Yea, I know the pain of people not shading from a different project, so much so that I added some code that will yell at the developer in case it's not setup correctly 😅. |
Decided to use the |
You're still initializing it far too late. |
You might want to rework the readme a bit. I just read
And took this as "as early as possible"(which would be |
As early as possible is preLaunch. Main entrypoint are fired after much of the game is already initialized and so after lots of mixins are already applied. |
Yes, but the wording makes it seem like the Client/Common entry points are fine(and it seems to work, at least initially). Most devs probably never had to touch the preLaunch point/are even aware that it's there. |
I think it's fine given I specifically say to use it. Regardless, you don't need to initialize it yourself anymore from the latest betas onwards. |
You shade it without relocating it, causing QuiltMC/quilt-loader#315.
You also crucially need to mergeServiceFiles or it will break on modern forge.
You also initialize it in a
ClientModInitializer
, that's way too late for most stuff. You don't seem to initialize it at all on forge. You need to do it in an IMixinConfigPlugin.The text was updated successfully, but these errors were encountered: