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

Update to 1.20.4 #7

Merged
merged 4 commits into from
Feb 1, 2024
Merged

Update to 1.20.4 #7

merged 4 commits into from
Feb 1, 2024

Conversation

unilock
Copy link

@unilock unilock commented Feb 1, 2024

This PR updates Gradle and the dependencies for Minecraft 1.20.4, and adds a @Unique annotation where one was missing.

(with the 1.20.2 version, I was running into a crash about an impossible cast when shift-clicking the crafting output, IIRC)

Copy link
Member

@Ampflower Ampflower left a comment

Choose a reason for hiding this comment

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

Seems fine, just a couple things I noticed

libs.versions.toml Outdated Show resolved Hide resolved
@@ -25,7 +25,7 @@
"quickbench.mixins.json"
],
"depends": {
"fabricloader": ">=0.7.2",
"fabricloader": "*",
Copy link
Member

Choose a reason for hiding this comment

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

Not exactly a needed change; if anything it probably should be vbump'd forward rather than requiring any version of loader

Copy link
Author

Choose a reason for hiding this comment

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

I agree that the change wasn't needed. It probably should've at least been bumped to >=0.14.0?

Copy link
Member

Choose a reason for hiding this comment

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

Yeah; it'll do for now at least, nothing required in a terribly specific version of the loader.

Copy link
Member

@Ampflower Ampflower left a comment

Choose a reason for hiding this comment

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

Looks good; I'll let the loader bit slide since it's not all that important

@Ampflower Ampflower merged commit 7fc0749 into Modflower:trunk Feb 1, 2024
1 check passed
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