-
-
Notifications
You must be signed in to change notification settings - Fork 86
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
Port a few modules up to 1.21 #375
Conversation
the deletions?? you're cooking?? |
Nope! Just using Mixin Extras |
This reverts commit d3d5e41.
Leaving disabled since I had to do some hacks to test.
Disabled a bunch of dependencies on res loader, make sure to fix this before merging
* @param overlay the overlay name | ||
* @param pack the pack | ||
*/ | ||
void putOverlay(@NotNull String overlay, ResourcePack pack); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ix0rai @TheGlitch76 Im not too sure about this. Should it be a ResourcePack
, where any pack can be part of the mutable pack, or does Consumer<ResourcePack>
make more sense where the MutablePack
implementation creates the pack in the way that it wants? Both options have their advantages.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have zero expertise in this area, it's really up to you...
@OroArmor can you compile a list of modules that you've ported in this PR? |
The list is:
Ported already (thanks glitch):
|
oh perf ok |
Also makes the DISABLE files per module, making it less of a diff to enable them.