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

#866 Feature: Choose Command #868

Merged
merged 7 commits into from
Dec 12, 2023
Merged

Conversation

arock475
Copy link
Contributor

Screen Shot 2023-11-24 at 3 14 43 PM Screen Shot 2023-11-24 at 3 14 59 PM

Copy link

netlify bot commented Nov 24, 2023

Deploy Preview for clembotdocs canceled.

Name Link
🔨 Latest commit f95f067
🔍 Latest deploy log https://app.netlify.com/sites/clembotdocs/deploys/656f5f7e16eb0f00081d254d

Copy link

Static type linting failed: Please see CI error output for details and fix the PRs typing

Copy link

isort linting failed: Please run poetry run isort . from the ClemBot.Bot folder

Copy link

Black linting failed: Please run poetry run black bot from the ClemBot.Bot folder

Copy link

Static type linting failed: Please see CI error output for details and fix the PRs typing

Copy link
Sponsor Member

@jkriste jkriste left a comment

Choose a reason for hiding this comment

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

Very minor changes needed, but looks good!

After the changes are committed and testing is done, this can be merged. 🎉

ClemBot.Bot/bot/cogs/choose_cog.py Outdated Show resolved Hide resolved
ClemBot.Bot/bot/cogs/choose_cog.py Outdated Show resolved Hide resolved
ClemBot.Bot/bot/cogs/choose_cog.py Outdated Show resolved Hide resolved
ClemBot.Bot/bot/cogs/choose_cog.py Outdated Show resolved Hide resolved
ClemBot.Bot/bot/cogs/choose_cog.py Outdated Show resolved Hide resolved
@jkriste
Copy link
Sponsor Member

jkriste commented Dec 12, 2023

Tested and approved. 🎉

@jkriste jkriste merged commit ee5736a into ClemBotProject:master Dec 12, 2023
1 check passed
@jkriste jkriste mentioned this pull request Dec 13, 2023
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