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

Begin work on updating to 1.19.3 #1142

Open
wants to merge 15 commits into
base: dev
Choose a base branch
from
Open

Begin work on updating to 1.19.3 #1142

wants to merge 15 commits into from

Conversation

ApionXD
Copy link

@ApionXD ApionXD commented Feb 12, 2023

The login process is fully working, including chunk encoding

I am not sure if this PR is suitable to merge in it's current state, as I made several opinionated changes that need to be rolled back (removed GitHub tasks, removed dependency on Linkstone, changed DSL from Kotlin to Groovy, removed all publishing info from gradle, removed the glowstone repo since it was down.) I also need to go over it with checkstyle.

Anyway, here's everything I've been working on the past few days

@CLAassistant
Copy link

CLAassistant commented Feb 12, 2023

CLA assistant check
All committers have signed the CLA.

@ApionXD
Copy link
Author

ApionXD commented Feb 13, 2023

All of the stupid stuff I did is fixed, I just need to fix some tests that are failing and checkstyle it.

@GeorgeV220
Copy link

Any news ?

Copy link
Member

@mastercoms mastercoms left a comment

Choose a reason for hiding this comment

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

LGTM, obviously this needs more work but it advances the work on the dev branch and can be built off of. Would like another set of eyes on this though since there were a lot of changes, and it's hard to understand all of them at once.

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.

None yet

4 participants