-
Notifications
You must be signed in to change notification settings - Fork 5
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
Move to unversioned python and add documentation #5
Conversation
MinecraftServer is Depreciated and will be removed in 2022-08
Fix Errors on Readme
Update to Python 3.9 on RHEL
Handle the expection so the template can recognize it as offline
Fix typo
Fix error
Add Trigger for Server Down
remove uuids
Fix so that old and new mcstatus is supported
Remove test python3.9 dependency and add example
Change name and update descriptions
Fix Template
Fixed typo
Fixed typo
Changed to Python3
# Get server instance | ||
server = MinecraftServer(sys.argv[2], int(sys.argv[3])) | ||
try: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I also made changes to implement the new mcstatus "JavaServer" class as well as the old one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should also fix #4
@@ -34,6 +34,13 @@ | |||
<name>Minecraft</name> | |||
</application> | |||
</applications> | |||
<triggers> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It also has a trigger that can notify you if a server goes offline.
I made these changes over the last few years while running a few minecraft servers for me and my friends.
Apparently i haven't made a PR yet.
The commits are somewhat crappy since I was debugging issues inside github and then pulling it onto the servers ^^