-
Notifications
You must be signed in to change notification settings - Fork 29
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
Remove dependency on Fabric API #10
Comments
It only depends on fabric-lifecycle-events-v1 submodule now so it should be simpler to jar in jar all required dependencies |
The fabric-api is required, fabric-api not only provides some server lifecycle events, but also the registry sync and some other patches. The test envirionment
If you modify the
You will get:
If we try to fix this and modified the code to
You still get:
If you comment out this
You will get:
Also, you can trigger this manually by issue the command |
I found this pr in The However, this mixin is deleted in branch |
Extracting individual modules out of fabric-api is for advanced users. It's not necessarily supported but to have the option is preferable. |
Currently, the only Fabric API features this mod uses are the lifecycle events. The dependency is so small that it should be able to be removed with just a couple mixins. Would you be open to doing that?
The text was updated successfully, but these errors were encountered: