Skip to content
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

Add new owner only commands - (un)sync & (re)(un)load #70

Merged
merged 10 commits into from
Sep 30, 2022

Conversation

shndowbots
Copy link
Contributor

@shndowbots shndowbots commented Sep 14, 2022

I've added the load, unload, and reload cog commands to the owner cog. This will allow the bot's owner to use cogs more efficiently without restarting the bot with every change.

Copy link
Owner

@kkrypt0nn kkrypt0nn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - please take a look at these potential changes ^^

cogs/owner.py Outdated Show resolved Hide resolved
cogs/owner.py Outdated Show resolved Hide resolved
cogs/owner.py Outdated Show resolved Hide resolved
cogs/owner.py Outdated Show resolved Hide resolved
cogs/owner.py Outdated Show resolved Hide resolved
cogs/owner.py Outdated Show resolved Hide resolved
cogs/owner.py Outdated Show resolved Hide resolved
cogs/owner.py Outdated Show resolved Hide resolved
cogs/owner.py Outdated Show resolved Hide resolved
cogs/owner.py Outdated Show resolved Hide resolved
@shndowbots
Copy link
Contributor Author

@kkrypt0nn Fixed and changed.

Copy link
Owner

@kkrypt0nn kkrypt0nn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please next time try to run the bot locally first before committing and pushing the changes, thanks.

cogs/owner.py Outdated Show resolved Hide resolved
cogs/owner.py Outdated Show resolved Hide resolved
cogs/owner.py Outdated Show resolved Hide resolved
cogs/owner.py Outdated Show resolved Hide resolved
cogs/owner.py Outdated Show resolved Hide resolved
cogs/owner.py Outdated Show resolved Hide resolved
cogs/owner.py Outdated Show resolved Hide resolved
cogs/owner.py Outdated Show resolved Hide resolved
cogs/owner.py Outdated Show resolved Hide resolved
@kkrypt0nn
Copy link
Owner

Thanks for the update, I will make the last changes tomorrow on my own and then merge it ⭐

@shndowbots
Copy link
Contributor Author

Thanks for the update, I will make the last changes tomorrow on my own and then merge it ⭐

Thank you!

@kkrypt0nn
Copy link
Owner

The API currently has some issues
image
Therefore will continue on the last commits and tests tomorrow

@kkrypt0nn kkrypt0nn changed the title Added load, unload, and reload cogs commands. Add new owner only commands - (un)sync & (re)(un)load Sep 29, 2022
@shndowbots
Copy link
Contributor Author

The API currently has some issues image Therefore will continue on the last commits and tests tomorrow

Alr

@kkrypt0nn kkrypt0nn merged commit 2d0f342 into kkrypt0nn:main Sep 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants