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

[WIP] Clean Up Test Mods #79

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

halotroop2288
Copy link
Contributor

@halotroop2288 halotroop2288 commented Jul 21, 2024

Changes

  • Drops support for Gradle 7 (Unimined is built against Gradle 8.8 now)
  • Renames the JUnit tests to fit with the new test mods (and not use underscores)
  • Organizes the new test mods into a new testing/minecraft subfolder
  • Combines the Cleanroom test mod into the 1.12.2-Forge-Fabric test mod and cleans up that project a lot.
    • MCP
      • Forge
      • Cleanroom
      • Fabric
      • Quilt
    • Legacy Yarn
      • Fabric
      • Quilt
  • Combines the 1.2.5 tests into one test mod
    • Calamus
      • Feather
      • Fabric
      • Quilt
    • MCP
      • ModLoader
        • How to use processResources in a ModLoader / old Minecraft Forge project
      • Minecraft Forge
  • Overhauls the beta 1.7.3 test mod, fixing the mapping and loader pairing
    • ModLoader will not work correctly on other mapping sets without modification
    • Added Minecraft Forge 1.0.7 to the modloader source set, it is optional.

Known issues

  • Cleanroom crashes whenever an entry is selected in the mod list. This is probably a Cleanroom bug.
  • Quilt does not work on any version before 1.0.0 as far as I can tell

@halotroop2288 halotroop2288 force-pushed the pr/testing-cleanup branch 2 times, most recently from 9c42da3 to 76c8f0d Compare July 21, 2024 06:50
@halotroop2288 halotroop2288 force-pushed the pr/testing-cleanup branch 6 times, most recently from db90b62 to 9c75d65 Compare July 30, 2024 01:25
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.

1 participant