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

Support for 1.20.3/4 #356

Merged
merged 7 commits into from
Mar 15, 2024
Merged

Support for 1.20.3/4 #356

merged 7 commits into from
Mar 15, 2024

Conversation

moehreag
Copy link
Contributor

@moehreag moehreag commented Dec 9, 2023

Tasks:

  • decide on which version this should get (please just edit it in before merging)
  • possibly create a new branch for this pr to target
  • figure out a solution for the test mods as they currently do not get loaded correctly because loom requires the mods block

Information: builds succeed, but gradle will still spit out errors about the javadocs as well as about not finding some of minecraft's classes. I don't know if these "errors" were present before, but the mod works fine.

@OroArmor
Copy link
Member

OroArmor commented Dec 14, 2023

Looks good from a cursory glance. Once I have time to fix 1.20.2, I'll merge that into this and then make a new 1.20.3 branch for 1.20.3/4 and then merge this

@ix0rai ix0rai added the t: refactor This proposes a refactor. label Dec 19, 2023
Co-Authored-By: Eli Orona <eliorona@live.com>
@OroArmor OroArmor merged commit 767263a into QuiltMC:1.20.3 Mar 15, 2024
2 checks passed
@FirstMegaGame4
Copy link

FirstMegaGame4 commented Mar 15, 2024

Why is the target branch named 1.20.3 and not 1.20.4? 1.20.3 is just 1.20.4 with a missing markDirty

@OroArmor
Copy link
Member

20.3 and 20.4 are compatible. if we updated on time, we would stay on the 20.3 branch and add support from there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t: refactor This proposes a refactor.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants