-
-
Notifications
You must be signed in to change notification settings - Fork 303
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
Introduced Utility Functions - Remove duplicate code snippets #356
Introduced Utility Functions - Remove duplicate code snippets #356
Conversation
d453427
to
d33c651
Compare
This PR has been rebased onto the latest main to integrate recent changes. |
I believe it's preferable to retain |
The change doesn't remove rcon-cli or adjust anything public, just provided a function wrapper so the config didn't need to be supplied a dozen times |
You're right, I thought it was replacing rcon-cli with RCON. Thank you for clarifying! |
@Phelms215 If you resolve the merge conflict, ready to merge! |
…pdate required detection
Removed redundant code with utility functions, Discord and Logging
…pdate required detection
Removed redundant code with utility functions, Discord and Logging
96693be
to
ce5e967
Compare
…nctions Introduced Utility Functions - Remove duplicate code snippets
Context
Choices
Test instructions
Checklist before requesting a review