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

Remove dependency and extendsFrom from ModModel DSL #150

Merged
merged 3 commits into from
Sep 1, 2024

Conversation

Technici4n
Copy link
Member

Fixes #119.

Removed because they do not work in IDEs. Even worse, sometimes they might appear to work! For example, before this PR, gradlew :jijtest:runData on the testproject works. Now, if you run Data from the IDE it will works because the jar file got built. However, if you remove coremod/build/libs/coremod.jar and try running with the IDE again... then it fails!

@Technici4n Technici4n added the breaking change This request is a breaking change label Aug 31, 2024
@neoforged-pr-publishing
Copy link

neoforged-pr-publishing bot commented Aug 31, 2024

  • Publish PR to GitHub Packages

@shartte
Copy link
Collaborator

shartte commented Aug 31, 2024

Should we add the evaluationDependsOn to the README?

@neoforged-automation neoforged-automation bot added the needs rebase This Pull Request needs to be rebased before being merged label Sep 1, 2024
@neoforged-automation
Copy link

@Technici4n, this pull request has conflicts, please resolve them for this PR to move forward.

@Technici4n
Copy link
Member Author

Should we add the evaluationDependsOn to the README?

For now I added a check for duplicates, which should already catch some cases.

@neoforged-automation neoforged-automation bot removed the needs rebase This Pull Request needs to be rebased before being merged label Sep 1, 2024
@Technici4n Technici4n merged commit 5bf2286 into main Sep 1, 2024
4 checks passed
@Technici4n Technici4n deleted the remove-mods-dependency branch September 1, 2024 21:42
@neoforged-releases
Copy link

🚀 This PR has been released as ModDevGradle version 2.0.25-beta.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change This request is a breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Decide the fate of dependencies in the mods block
2 participants