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

Modify the project structure & update various portions #151

Merged
merged 9 commits into from
Nov 12, 2022

Conversation

sschr15
Copy link
Contributor

@sschr15 sschr15 commented Nov 11, 2022

  • Forge test mod updated to match
  • Both test mods moved into main project structure
  • All modules share the same versions
  • Version management moved to gradle/libs.versions.toml
  • Gradle version updated to current latest release

Test mods and individual modules are less likely to fall out of sync with each other now, and with one single version location management, it is likely a lot easier to update versions in the future.

@sschr15 sschr15 marked this pull request as draft November 12, 2022 04:20
@sschr15 sschr15 changed the title Update Forge test mod to 1.19.2 Modify the project structure & update various portions Nov 12, 2022
@sschr15 sschr15 marked this pull request as ready for review November 12, 2022 05:53
Copy link
Owner

@natanfudge natanfudge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A very needed change, thanks a lot. After this I'll add support for refreshVersions as well.
Please also add to TESTING.md information about how the test mods can be used by uncommenting lines in fabric/build.gradle and forge/build.gradle now.

TestForgeMod/build.gradle Show resolved Hide resolved
TestForgeMod/gradle.properties Outdated Show resolved Hide resolved
build.gradle Show resolved Hide resolved
build.gradle Show resolved Hide resolved
fabric/build.gradle Outdated Show resolved Hide resolved
forge/build.gradle Outdated Show resolved Hide resolved
forge/build.gradle Outdated Show resolved Hide resolved
forge/build.gradle Outdated Show resolved Hide resolved
fabric/build.gradle Outdated Show resolved Hide resolved
@natanfudge
Copy link
Owner

natanfudge commented Nov 12, 2022

I seem to be getting this warning whenever I build when the test mod is included with modRuntimeOnly:

> Configure project :TestFabricMod
Found existing cache lock file, rebuilding loom cache. This may have been caused by a failed or canceled build.

Can we do anything about this?

@natanfudge
Copy link
Owner

natanfudge commented Nov 12, 2022

It seems like loom just doesn't work very well when referencing other subprojects. Don't add that stuff for now (keep TESTING.md as-is as well).
See FabricMC/fabric-loom#752

Copy link
Owner

@natanfudge natanfudge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

def cf = libs.versions.overrides.curseforge.get()
def mr = libs.versions.overrides.modrinth.get()

ext.curseforge_mc_version = cf == 'mc' ? minecraft_version : cf
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice

@natanfudge natanfudge merged commit 034e8a2 into natanfudge:1.19 Nov 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants