-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
[Other]: Remove the MelonLoader net6preview build #40
Comments
It may indeed confuse users, but I think it is necessary since v6.0.0 Alpha has been released on GitHub. |
It is completely unsupported in the community, and no one is or should be using it. Otherwise, I suggest to rename the UE build to specify the ML version it targets |
If I remove this because the community doesn't support it, then I must also remove the old version of BepInEx, otherwise it will not add up.
Is it mentioned in the README, but that is not enough? |
I think it would be better to show that |
That's also fine, as long as the user is aware they're using the wrong version. We get a lot of reports with people attempting to run the net6preview build on the wrong version. I also suggest implementing a check before initializing the mod, as it often results in initializing on the wrong version and throwing a ton of errors.
The release is very unstable and no one should be using it. We are actively discouraging people from using this release (which almost never happens regardless). The release was only meant as a test for the full release of CoreCLR and a warning for mod developers to update their mods before the full release |
This is a problem of user understanding, and I feel that it's wrong to try to solve the problem only by the developers, including myself(There is even a possibility of using
The initialization process shouldn't have even been called because it failed at the point of loading the UE DLL.
As a mod developer, I understand that, but the user may not necessarily understand it. The best solution would probably be to remove (or empty) the |
If |
I don't think removing the version is a good idea, but a warning for the user would be great. Maybe it would be worth removing the Either way, thanks for the consideration |
I think that even if v1.0.0 of ML is released, this problem willn't be solved as long as users can DL either |
Describe the issue
Since this build was only used for indev 0.6 versions, it is no longer needed and only serves as a confusion for the user
The text was updated successfully, but these errors were encountered: