-
Notifications
You must be signed in to change notification settings - Fork 24
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
1.18.1 support #58
1.18.1 support #58
Conversation
Note: In this commit, ExtensionPoints are currently broken due to a FML API change.
Odd... I'll give it a look. I didn't bring over the newer Gradle version from my Eclipse build environment so I wonder if that might be the cause. |
@skedgyedgy I ran into the same problem but the issue should be resolved now. I updated the PR's Gradle version and the mod now builds properly from the command line. |
@@ -85,7 +85,7 @@ public void preInit() { | |||
ModLoadingContext context = ModLoadingContext.get(); | |||
context.registerExtensionPoint(IExtensionPoint.DisplayTest.class | |||
, () -> new IExtensionPoint.DisplayTest( | |||
() -> FMLNetworkConstants.IGNORESERVERONLY, | |||
() -> "ANY", |
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.
use a constant
@@ -104,7 +104,7 @@ public void preInit() { | |||
ModLoadingContext context = ModLoadingContext.get(); | |||
context.registerExtensionPoint(IExtensionPoint.DisplayTest.class | |||
, () -> new IExtensionPoint.DisplayTest( | |||
() -> FMLNetworkConstants.IGNORESERVERONLY, | |||
() -> "ANY", |
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.
use a constant
Adds support for Minecraft 1.18.1. Initial tests so far have worked as expected. I've been using this patch for a while with several people using a MC 1.18.1 server and Mumble 1.3.4 and so far it has worked fine.
Changes made in this PR: