Dinnerbone / mcstatus

A Python class for checking the status of an enabled Minecraft server
http://dinnerbone.com/minecraft/tools/status/
1.11k stars 146 forks source link

Rename server classes #219

Closed ItsDrike closed 2 years ago

ItsDrike commented 2 years ago

As was already discussed on the discord server, we decided that it would be a good idea to rename MinecraftServer to JavaServer instead, since the reason for naming it MinecraftServer was purely because this library was made before bedrock edition even existed so there was no confusion, however now, that's not the case anymore and it's very odd to have MinecraftServer and MinecraftBedrockServer classes, with MinecraftServer being used for the java version.

This also drops the Minecraft prefix from these classes (MinecraftBedrockServer becomes BedrockServer) since we've decided that there's no point in keeping it either as in most cases, it will be obvious what these names are referring to, and if there was a conflict somewhere, people can always use an import as statement (from mcstatus import JavaServer as MinecraftJavaServer)

To preserve backwards compatibility, I've added simple classes which inherit from these new ones, having the original names. These classes will produce deprecation warnings if used.

ItsDrike commented 2 years ago

I hate this (wrong repo again)