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

Fix potential issue with Boat Data #1903

Merged
merged 1 commit into from
Feb 8, 2021

Conversation

bundabrg
Copy link
Collaborator

@bundabrg bundabrg commented Feb 6, 2021

I believe it was Abraham Lincoln who said that the smallest change can have the greatest impact; that being the change is an oft once in a life time opportunity and it is through trying times that hero's are born, nay forged by the fires of the challenge thrust upon them. Or it might not have been him but lets not let truth get in the way.

Gentlemen, such a time has come upon us and requires the utmost of care to solve as well as the boldest of moves. Risks must be taken but ultimately we will prevail because the human spirit is strong.

I put it out that this change is of critical importance and the very act of performing this merge may very likely be the greatest thing ever to be achieved by whomever is lucky enough to do so. You can rest your head, weary as it may be, knowing that your job is done and that it is only downhill from here.

@Hellohi3654
Copy link
Contributor

lol would be funny if this completely fixed boats

@Camotoy
Copy link
Member

Camotoy commented Feb 6, 2021

Great change, o quoter of Lincoln, but it would be rather helpful if you updated your mappings submodule. xd

@bundabrg bundabrg force-pushed the fix/conserve-brackets branch from 69d3597 to 20c73c8 Compare February 7, 2021 06:21
@bundabrg
Copy link
Collaborator Author

bundabrg commented Feb 7, 2021

It has been sorted.

@Camotoy
Copy link
Member

Camotoy commented Feb 7, 2021

Have you tested and confirmed that boats still operate normally on 1.16.100 and 1.16.200?

@bundabrg
Copy link
Collaborator Author

bundabrg commented Feb 8, 2021

yep.

@Camotoy Camotoy merged commit 66fcd87 into GeyserMC:master Feb 8, 2021
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