Skip to content

Update fabric.mod.json and neoforge.mods.toml to match the version supported - #86

Closed
Novampr wants to merge 2 commits into
Incendo:masterfrom
Novampr:master
Closed

Update fabric.mod.json and neoforge.mods.toml to match the version supported#86
Novampr wants to merge 2 commits into
Incendo:masterfrom
Novampr:master

Conversation

@Novampr

@Novampr Novampr commented Jul 13, 2025

Copy link
Copy Markdown

Causes issues if you attempt to launch on lower versions, ideally Fabric and NeoForge should catch this beforehand, but can only do so with the right metadata.

@jpenilla

jpenilla commented Jul 24, 2025

Copy link
Copy Markdown
Member

You've tested that both platforms error on 1.21.5 startup? If only some features are broken we don't want to be so strict.

@Novampr

Novampr commented Jul 27, 2025

Copy link
Copy Markdown
Author

Yeah, the mod crashes with a missing Minecraft class on start, on 1.21.6 and onwards, no issues were found on 1.21.6

@jpenilla

jpenilla commented Jul 28, 2025

Copy link
Copy Markdown
Member
Screenshot 2025-07-27 at 5 17 21 PM Can you provide a stacktrace for each platform? By itself it doesn't crash but I want to be sure it's the command manager itself and not something specific to your command code.

@Novampr

Novampr commented Jul 28, 2025

Copy link
Copy Markdown
Author

Heres the logs of the stacktrace, looks to be the command manager: https://mclo.gs/pAYcecR

@jpenilla

jpenilla commented Jul 28, 2025

Copy link
Copy Markdown
Member

It looks like that class was added in 1.21.5, so unless something else is broken that can be the minimum Fabric version - for Neo it seems like >=1.21.6 is correct

@jpenilla jpenilla closed this in 4a4bf39 Oct 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants