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 snapshot 24w03a (1.20.5) #3601

Closed
wants to merge 1 commit into from

Conversation

Outfluencer
Copy link
Collaborator

if you spot bugs please report

@Outfluencer
Copy link
Collaborator Author

i repackaged the packets as its not very nice to have that much packets flat

@Outfluencer
Copy link
Collaborator Author

i hope i can help you a bit out with this pr

@md-5
Copy link
Member

md-5 commented Jan 20, 2024

i repackaged the packets as its not very nice to have that much packets flat

Makes the PR extremely difficult to review. Please only include the changes necessary for snapshot support

@Outfluencer
Copy link
Collaborator Author

okay i will revert the renaming and create a new pr

@md-5
Copy link
Member

md-5 commented Jan 20, 2024

Thanks

@Outfluencer
Copy link
Collaborator Author

look at #3602

@Janmm14
Copy link
Contributor

Janmm14 commented Jan 20, 2024

As packets are part of api, renaming could break bunch of plugins using unsafe api

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.

3 participants