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

Upgrade to 1.20.0 #89

Merged
merged 2 commits into from
Jun 10, 2023
Merged

Upgrade to 1.20.0 #89

merged 2 commits into from
Jun 10, 2023

Conversation

LostLuma
Copy link
Collaborator

@LostLuma LostLuma commented Jun 8, 2023

Hey, since this is one of the few mods I use I thought I'd update it to work in 1.20.0!

  • Update HudInfoRenderer for 1.20.0
  • Upgraded dependencies (Gradle, Loom, ModMenu, Cloth Config)
  • Added machete to the build script to compress output
  • Add Minecraft as a dependency so the game will not start with eg. Minecraft 1.19.4
  • Removed LazyDFU from dev env, as it's no longer updated thanks to performance improvements from Mojang

As I've not worked on a Fabric mod using Quilt mappings before I'm not sure whether I've upgraded the build.gradle the "correct" way, however it does build the mod perfectly fine.

Ensure Loader errors when used on an earlier Minecraft version.
@juliand665
Copy link
Owner

Oh, you are an absolute godsend—thank you for doing this again! I'll try to take a look tomorrow, then merge & release :)

@arnokeesman
Copy link

I get a crash in singleplayer, crash report latest.log. Also tried with modmenu and clothconfig installed, didn't help. Though they don't seem related to the issue looking at the logs anyway.
If there's anything else I can provide to help debug this, don't hesitate to ask.

@LostLuma
Copy link
Collaborator Author

LostLuma commented Jun 9, 2023

That crash is what this PR fixes, you're running the Dynamic FPS 2.2.0 for 1.19.4.
If you'd like to test out this PR for 1.20.0 I've uploaded a locally built jar here: https://files.lostluma.net/dynamic-fps-2.3.0.jar

@arnokeesman
Copy link

okay yeah that's totally my bad, did grab from your repo, just not the correct branch, woops

@juliand665 juliand665 merged commit 4924876 into juliand665:main Jun 10, 2023
@juliand665
Copy link
Owner

Just got around to trying it—works like a charm! Thank you so much for this <3

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.

None yet

3 participants