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
More system properties #1515
More system properties #1515
Conversation
Signed-off-by: TheKodeToad <TheKodeToad@proton.me>
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.
i like this, just have a few suggestions
If this could get approved before Oct 26th 2023, I can happily support this within CraftPresence's PrismLauncher pack implementation. As Oct 26th is the planned release date for the upcoming v2.2.4 patch for my mod |
Signed-off-by: TheKodeToad <TheKodeToad@proton.me>
is it still a good idea to support MultiMC properties? maybe it makes sense as we are a fork |
@TheKodeToad Personally, I think it would be nice, at least to preserve compatibility with its variables, if other mods decide to utilize those variables. |
Signed-off-by: Sefa Eyeoglu <contact@scrumplex.net>
(CraftPresence already reads the instance.cfg file but adding multimc's props is a small optimisation that's already implemented)