Closed TelepathicGrunt closed 9 months ago
I concur. This seems within the purview of a mod like In Control! as opposed to the modloader.
Yeah I have no idea why these were controls in the first place, they're extremely old. They can be nuked next BC window.
And in my opinion, mod loaders should not fix vanilla bugs.
It depends on the bug, some Vanilla bugs can adversely effect modding environments quite drastically. Fixing these at the Loader level makes sense. However that isn't relevant in this case as this isn't a bug fix, this is an additional feature which falls in a category really outside our purview.
just btw
Also, this config isn’t even fixing any bugs. They are just… there… for no reason that I can see lol
I came across this in the server config file for Neoforge which was in inherited from Forge.![image](https://github.com/neoforged/NeoForge/assets/40846040/b880e44a-12fa-4f1a-b648-88bfe5d4d6f5)
In my opinion, this is not the domain of a modloader API. Rather this zombie villager baby chance should be handled by a separate mod made by someone else that Neoforge loads just like any other mod when in mod folder. There's no reason for this config option to be in Neoforge itself.
Thoughts?
Code: https://github.com/neoforged/NeoForge/blob/0974ecc143f449087139a6923aa3f0823dce421d/src/main/java/net/minecraftforge/common/ForgeConfig.java#L26-L27