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

Multi-version support (1.16.5, 1.18.1, 1.18.2) #61

Open
wants to merge 29 commits into
base: master
Choose a base branch
from

Conversation

Combustible
Copy link
Contributor

I don't honestly expect this pull request to ever get merged (and maybe it shouldn't), but I wanted to submit it anyway.

I've attached my fork's master branch to this PR... so for better or worse as I continue to push new commits to it those commits will make their way here too. If it's annoying you can close this.

I got tired of patching up random reflection problems - so I've refactored things into NMS version adapters that are loaded at startup. This creates a fair amount of code duplication - but also means that new versions don't require breaking / refactoring older versions.

Paper's new Paperweight build system (since 1.17) makes this much easier to maintain than it used to be.

Combustible and others added 29 commits October 8, 2020 06:33
Signed-off-by: Byron Marohn <[email protected]>
Signed-off-by: Byron Marohn <[email protected]>
Signed-off-by: Byron Marohn <[email protected]>
Signed-off-by: Byron Marohn <[email protected]>
Signed-off-by: Byron Marohn <[email protected]>
Signed-off-by: Byron Marohn <[email protected]>
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.

4 participants