-
Notifications
You must be signed in to change notification settings - Fork 277
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
Mod install reasons #1617
Labels
enhancement
New feature or request
Comments
Added bonus would be to enable users to set reasons, for example: setting a mod as being a dependency of another mod, rather than an explicit install. |
This is a good idea, but should likely be roped in with modpack updates. Not a huge change as long as we don't break the MultiMC zip format. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Role
I frequently add a few mods onto mod packs
Suggestion
Track mod install reason: Modpack, explicit, dependency
Benefit
To improve update checking
This suggestion is unique
You may use the editor below to elaborate further.
I'd like to see PolyMC record a "reason" for why a mod is added to an instance, similar to linux package managers. Presently, this would have little immediate impact but would enable other requested features.
Reasons could be used to inform update checking: Modpack mods could be skipped over in lieu of checking for a new version of the pack itself. If the modpack removes the mod or changes it's version in an update, the update checker can assume it should update/remove the mod accordingly. #65 #324 #1591
Explicit mods could be handled the way all mods presently are, and dependencies could be checked either the same, or only when their dependents have updates.
Regardless, if a mod is removed, or at the user request, mods with the dependency reason should be checked for dependents that have a different install reason. If there are none, suggest removal.
Reasons could also be displayed in the mod list, to assist users in managing their own additions to packs.
The text was updated successfully, but these errors were encountered: