Heavily refactored the code, added support for Minecraft 1.8 all the way to 1.16.5, reworded messages to make them clearer, fixed some bugs, and more…

[akemin-dayo]

This is… a pretty big pull request.

Here's a list of the changes (somewhat adapted from the commit messages):

• Maven resource filtering is now used for the version numbers found in plugin.yml and bungee.yml.
  • This means that simply updating the project version in pom.xml will also automatically update the corresponding values in plugin.yml and bungee.yml!
  • Unfortunately, this does not work for Velocity's @Plugin annotation. That will still need to be manually updated for every release/build. :/
• Correctly get the true Bukkit (or Spigot) server root directory instead of incorrectly assuming that the world container root directory will always be the same as the server root directory.
  • This fixes a bug where GeyserUpdater will fail to find spigot.yml if the server owner has a world container set (-W argument at startup, and/or world-container configuration value set in bukkit.yml).
• Removed redundant log messages for Spigot and BungeeCord, as the plugin loader already prints the relevant messages.
• Target Bukkit API 1.8 to maximise compatibility with any Paper/Spigot/CraftBukkit/etc. server that could possibly be running Geyser with ViaVersion. Reflection is now used to invoke the Spigot restart() API method call at runtime to prevent older server versions (or anyone still using CraftBukkit for… some reason) from crashing.
• Renamed GeyserVeloDownload to GeyserVelocityDownload for naming consistency with other classes.
• Heavily refactored the CheckBuildNum class to dramatically decrease the amount of unnecessarily duplicated logic. Also modified the build number comparison logic so that GeyserUpdater handles build numbers higher than upstream correctly.
• Cleaned up unused and wildcard imports.
• Split up the CheckBuildNum class for each individual server platform to avoid runtime import resolution failures due to needlessly attempting to import classes that ultimately go unused and may not even exist on some server platforms. This change also makes makes GeyserUpdate fully function on every Minecraft version between 1.8 and 1.16.5!
• Changed almost all of the messages in GeyserUpdater to have significantly clearer wording. Some log severity levels have been modified to better suit the accompanying message. This commit does change config.yml, but it does not update the configuration version since there were no actual new options added. No functionality changes were made.
• Reworded README.md to make some things clearer and to show the supported Minecraft versions.

Let me know if there are any changes you'd like me to make!

(I'm also in your Discord now, if you'd like to ask any questions outside of GitHub's UI ;P)

[Konicai]

Closing this because of heavy conflicts with v1.4.0... majority of changes will need to be ported. Great stuff though.

[akemin-dayo]

Yeah, I'll port the changes over to the 1.4.0 branch and open another pull request when it's done, as we discussed in Discord ;P