Skip to content
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

Some GitHub Releases mods still fail to match #47

Closed
aschoettler opened this issue May 7, 2022 · 6 comments
Closed

Some GitHub Releases mods still fail to match #47

aschoettler opened this issue May 7, 2022 · 6 comments
Labels
bug Something doesn't work as expected priority I'm working on it, or will be soon

Comments

@aschoettler
Copy link

Here are a few mods I tried to download from github repositories which are not available on CurseForge. They all have 1.18.2 versions available but none of these were found by ferium. In each case, the version 1.18.2 appears in the title of the release, the filename of a jar file, or both.

@theRookieCoder
Copy link
Collaborator

theRookieCoder commented May 7, 2022

None of these mods have the mod loader specified. This is what the mod loader check override is for (see here)

@theRookieCoder theRookieCoder changed the title Examples of github release mods which failed to match for 1.18.2 Some GitHub Releases mods still fail to match (for 1.18.2) May 7, 2022
@JustSimplyKyle
Copy link
Contributor

Well, I think the problem this user has is that it doesn't download the correct mod version. E.g.
In 1.18, If I disable both modloader and version check for syncmatica, It'll still download the 1.16.5 version of it.

@aschoettler
Copy link
Author

Thank you for the response. As @KyleUltimate pointed out, disabling the mod loader check leads ferium to download an incorrect version for 3 of these 6 mods. For senseiwells/EssentialClient, senseiwells/ChunkDebug, and End-Tech/syncmatica, it found and downloaded the 1.16.5-compatible file from the correct GH release where the release also lists the 1.18.2 jar file in the assets.

@theRookieCoder
Copy link
Collaborator

Ah I see, I apologise for the misunderstanding

@theRookieCoder theRookieCoder reopened this May 8, 2022
@theRookieCoder theRookieCoder added bug Something doesn't work as expected priority I'm working on it, or will be soon labels May 8, 2022
@theRookieCoder theRookieCoder changed the title Some GitHub Releases mods still fail to match (for 1.18.2) Some GitHub Releases mods still fail to match May 8, 2022
@theRookieCoder
Copy link
Collaborator

theRookieCoder commented May 8, 2022

Ah I figured it out, the problem is that the game version check also checks the release name. This won't work when there are multiple versions in the same release. I guess we could remove that check and make it more strict, and the user can just disable the check if its too strict

@theRookieCoder
Copy link
Collaborator

theRookieCoder commented May 9, 2022

Apologies for not releasing this yet! I'm currently working on a major refactor and I can only release after #52 is resolved. It is available on crates.io so you can update Ferium from there if needed The action randomly worked again

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something doesn't work as expected priority I'm working on it, or will be soon
Development

No branches or pull requests

3 participants