Skip to content

Conversation

@R00tB33rMan
Copy link

BetterRTP currently:

  • Does not locally compile
  • Has various exceptions when using Folia

This PR resolves all of these issues. I've extensively tested these modifications on a 200-400 player production instance without problems (even performance-wise).

@ZepsiZola
Copy link
Contributor

@SuperRonanCraft
What do you think of this PR?

@SuperRonanCraft
Copy link
Collaborator

Coming back into things, this update seems huge! Might have worked out some kinks I've been wanting to tackle. I will totally take a look as I know some maven repos are not compatible in some versions, so I will do some testing before accepting this PR!

@R00tB33rMan
Copy link
Author

Coming back into things, this update seems huge! Might have worked out some kinks I've been wanting to tackle. I will totally take a look as I know some maven repos are not compatible in some versions, so I will do some testing before accepting this PR!

Sounds good to me! I'll rebase this PR that way it's ready to merge

@R00tB33rMan
Copy link
Author

Hey @SuperRonanCraft, any changes needed or is this all set?

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