-
-
Notifications
You must be signed in to change notification settings - Fork 37
Add Modrinth project files support #580
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
Conversation
Uses the same syntax as curseforge (@/path/to/modrinth-mods.txt)
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.
Thanks!
Just need to investigate the unit test failure.
It was a stubs issue with the test failure, I think I fixed it. |
Still failing. Please make sure you're running tests locally before pushing such as with
|
Figured out the issue and tests pass when running locally for me. Sorry for the back and forth on this 😭 |
src/test/java/me/itzg/helpers/modrinth/ModrinthCommandTest.java
Outdated
Show resolved
Hide resolved
src/test/java/me/itzg/helpers/modrinth/ModrinthCommandTest.java
Outdated
Show resolved
Hide resolved
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.
Thanks!
Now released in https://github.com/itzg/mc-image-helper/releases/tag/1.45.0 Can bump the version at As part of itzg/docker-minecraft-server#3506 |
Uses the same syntax as Curseforge:
@/path/to/modrinth-mods.txt
. Added tests to make sure it is working as expected. I tried to follow how it was done for Curseforge as closely as possible to keep things consistent. I will make a separate PR to update the docs at itzg/docker-minecraft-server/docs/mods-and-plugins/modrinth.mddocs PR: itzg/docker-minecraft-server#3506