-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add ViaMCP for MCP Reborn #31
base: main
Are you sure you want to change the base?
Conversation
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.
People just shouldn't use MCP in modern versions as proper mod loading systems like Fabric or NeoForge exists. Making a MCP based mod usually means not updating to newer versions since you don't know which modifications you made to the codebase and also you aren't able to install mods like Sodium/ImmediatelyFast which are quite important for proper gameplay in terms of performance (at least in my opinion).
I'll review/merge your changes later, maybe we could also add 1.16.5 since many people asked for this? Also what 1.20 is this? The folder should have the exact name.
1.20.2 is the version I'll change that for you |
1.20.2 is random, it should be either 1.20.6 or 1.21 |
Adding version 1.18.1 may be good? |
1.18.1 is even more random imo, 1.18.2 can be done if people want it but I generally don't recommend MCP for modern versions (and also not versions below 1.20.4) |
It's crazy that people still use MCP instead of Fabric. It's also crazy that there are people who call Yarn "ass" (it's not) meanwhile Searge having 5000 useful functions not mapped, and Yarn has most of them remapped.
|
holy yap |
it's crazy that ViaMCP still does not have support for MCP Reborn