Update to released API's NeoForgeExtension.setVersion()#1
Update to released API's NeoForgeExtension.setVersion()#1kylev wants to merge 1 commit intoAlmostReliable:mainfrom
Conversation
This API changed from the unstable releases.
|
While it's understandable to make this toolchain work with the newest MDG version, I don't really get the reason behind it. Why do you need that to set up ME Requester? The mod uses a fixed version and I can set it up just fine. That must mean you updated the version yourself for some reason? And why do you execute the test task? The mod doesn't use tests. |
|
I encountered a bug in merequester while playing ATM-10 and went to reproduce it, and found the tooling there outdated and a bit clunky to get working on my machine. I encountered this upstream problem as a result. Note: this currently means all downstream consumers of almostgradle 1.1.1 must use pre-release versions of ModDevGradle < 2.0.64-beta. |
|
As mentioned in chat, the upstream API change creates some hassle. Depending how this is merged or released, others' builds may break. One suggestion is to release it as 1.2.0 "MDG is out of beta"-release to prevent projects like lootjs that use the |
|
Superseded by #2. |
Proposed Changes
Fix build against now-stabilized ModDevGradle API.
Additional Context
I encountered a runtime error while setting up a AlmostReliable/merequester dev environment:
While trying to run the almostgradle
testtask, the 2.0.80 version of moddevgradle caused a compilation error:The API changed in neoforged/ModDevGradle#200 which became 2.0.64-beta.