-
-
Notifications
You must be signed in to change notification settings - Fork 187
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
Update to simpler mod loading from latest FML #774
Update to simpler mod loading from latest FML #774
Conversation
Last commit published: ec5b1f2723752abee6c74be10ad6eb69ae9e3c6e. PR PublishingThe artifacts published by this PR:
Repository DeclarationIn order to use the artifacts published by the PR, add the following repository to your buildscript: repositories {
maven {
name 'Maven for PR #774' // https://github.com/neoforged/NeoForge/pull/774
url 'https://prmaven.neoforged.net/NeoForge/pr774'
content {
includeModule('net.neoforged', 'neoforge')
includeModule('net.neoforged', 'testframework')
}
}
} MDK installationIn order to setup a MDK using the latest PR version, run the following commands in a terminal. mkdir NeoForge-pr774
cd NeoForge-pr774
curl -L https://prmaven.neoforged.net/NeoForge/pr774/net/neoforged/neoforge/20.5.0-alpha.1.20.5-rc1.20240418.193017/mdk-pr774.zip -o mdk.zip
jar xf mdk.zip
rm mdk.zip || del mdk.zip To test a production environment, you can download the installer from here. |
b510e4e
to
d4d637c
Compare
d4d637c
to
c6196a9
Compare
src/main/java/net/neoforged/neoforge/internal/CommonModLoader.java
Outdated
Show resolved
Hide resolved
c6196a9
to
f845000
Compare
return registriesLoaded; | ||
} | ||
|
||
protected static void begin(Runnable periodicTask) { |
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.
Comment when this is called? Or when it should be called?
Especially to explain why we have begin
and load
and why are they split?
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 added a vague comment - the current separation is only an artifact of how we do part of the mod loading inside a reload listener (which introduces its own set of problems btw...). I don't really want to encode it because we have discussed a potential rework of the client mod init timings.
} | ||
|
||
protected static void load(Executor syncExecutor, Executor parallelExecutor) { | ||
Runnable periodicTask = () -> {}; // no need to pass something else for now |
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.
Why?
See neoforged/FancyModLoader#110.