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
Added version check for neoforge filter #1658
Conversation
Signed-off-by: Trial97 <alexandru.tripon97@gmail.com>
I don't think this version comparison works with snapshots (and I wouldn't actually be surprised if they are supported in future at this point lol) |
Signed-off-by: Trial97 <alexandru.tripon97@gmail.com>
It looks like that would not be a problem the Version class can handle the snapshots stuff. |
I'm talking about the [year]w[week][a-z] format |
Signed-off-by: Trial97 <alexandru.tripon97@gmail.com>
Ok, I added a more strict condition: the forge is added just for 1.20.1. |
Co-authored-by: TheKodeToad <TheKodeToad@proton.me> Signed-off-by: Alexandru Ionut Tripon <alexandru.tripon97@gmail.com>
Made the neoforge loaders filter dependent on the instance version.
Currently, 1.20.1 is compatible with the forge, but with 1.20.2 the compatibility is dropped.