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

Java Edition 1.21.2 / 1.21.3 support #5106

Merged
merged 60 commits into from
Nov 5, 2024
Merged

Java Edition 1.21.2 / 1.21.3 support #5106

merged 60 commits into from
Nov 5, 2024

Conversation

Camotoy
Copy link
Member

@Camotoy Camotoy commented Oct 25, 2024

See here for a general idea of what needs to be addressed: #5100

Camotoy and others added 20 commits July 28, 2024 17:22
* Work on new tag system, still largely unfinished

* Continue work on new tag system, closer to testing now

* Add holderset class for geyser

* My IDE did some stuff it shouldn't have done

* Add is method for HolderSets to TagCache

* Add some documentation

* Fix build and slightly adjust documentation

* Fix some issues, still broken

* Fix more issues with registering vanilla tags, works better now

* Fix all issues with vanilla and non-vanilla tags, fix documentation a bit.

* Small cleanup

* Use IllegalArgumentException instead of asserting and make TagRegistry#getVanillaTags return original map

* Generics - still untested

* Remove different lookups for vanilla- and non-vanilla tags

* Add toNetworkId and fromNetworkId methods to JavaRegistryKey, simplify TagCache

* More cleanup, add documentation

* Fix registry loading

* Rename HolderSet to GeyserHolderSet and address reviews by camotoy

* Make holder set reader method not throw an exception, clean up

* Make holder set reader warn when failing to read

* Add comment to make clearer why a cast is make

---------

Co-authored-by: chris <[email protected]>
@Camotoy Camotoy added the PR: Needs Testing When a PR needs testing but is currently not under review label Oct 25, 2024
@dom111
Copy link

dom111 commented Nov 1, 2024

Hey @Camotoy,

Thanks for your hard work on this, great to see the progress happen!

I've been testing this PR branch on my server, apologies if this is not the right place to ask, but would you like any issues we encounter to be logged somewhere (probably not yet, but when you're working towards an RC maybe?)

Thanks!

Dom

@Camotoy
Copy link
Member Author

Camotoy commented Nov 1, 2024

Hi! We are nearing a stable build. Until we do, you are free to log any issues you find here.

@onebeastchris onebeastchris linked an issue Nov 2, 2024 that may be closed by this pull request
@Camotoy Camotoy merged commit 6983095 into master Nov 5, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: Needs Testing When a PR needs testing but is currently not under review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The bedrock version cannot normal use the scaffolding
5 participants