Skip to content

Hard Fork of Project Structure - #151

Open
Thatsmusic99 wants to merge 4 commits into
v6from
dev/paper-hard-fork
Open

Hard Fork of Project Structure#151
Thatsmusic99 wants to merge 4 commits into
v6from
dev/paper-hard-fork

Conversation

@Thatsmusic99

Copy link
Copy Markdown
Collaborator

Information

Splits up AT's project into four different submodules:

  • Common, where all common non-platform specific code lives
  • Spigot, where all Spigot-specific code lives
  • Paper, where all Paper-specific code lives
  • Hybrid, where both platforms are put into one JAR file, but the plugin decides which codebase is more appropriate to use in a certain scenario

This has become a necessity with Paper's hard forking from Spigot, where it's become a lot harder to tell which code belongs to Paper and which is only available on Spigot. It also makes it easier to catch compilation errors for either platform.

This isn't a perfect split at the moment - the point of this MR is to just get the job done, with further refinement being open down the line.

Also closes #150 as part of this - needs testing

Prior to Merging

  • Has the PR been tested?
    • Needs Spigot and Hybrid tested on Spigot server
    • Needs particle-vanish testing
    • Needs new world structure testing for /tpofflh which has been disabled in Spigot
  • Do you consent to GitHub Copilot also reviewing your changes? An actual developer will review your changes first.

Signed-off-by: Thatsmusic99 <25277367+thatsmusic99@users.noreply.github.com>
Signed-off-by: Thatsmusic99 <25277367+thatsmusic99@users.noreply.github.com>
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.

Change particle toggle from config option to a permission

1 participant