-
Notifications
You must be signed in to change notification settings - Fork 9
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
feat: introduce aws to minectl #210
Conversation
List:
|
Delete:
|
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.
Hi @cpanato, huge thanks for this work! ❤️
Just some small comments,
But could you update the README.md file to:
- A AWS badge
- The Credendtials part
- And an example config file in the config folder for whatever Server edition you want
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.
Just two minor things, and could you submitt your config file two? So we have this as an example!
Thank you so much!
1da0160
to
a68c6cf
Compare
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.
Hi,
i added the UDP / TCP Support for bedrock server and Monitoring and Rcon support.
I hope its okay? I have the code here if its easier to than my suggestions..
Other then that it works really well! Thanks!
Signed-off-by: Carlos Panato <ctadeu@gmail.com>
There is a merge conflict, but i will fix this! Thanks @cpanato for this huge help! |
Add initial implemenation for AWS provider
future work: