Skip to content

Fix !clean users command to handle more than one user#2178

Merged
Xithrius merged 3 commits into
python-discord:mainfrom
Autonymic:fix/clean-users
May 27, 2022
Merged

Fix !clean users command to handle more than one user#2178
Xithrius merged 3 commits into
python-discord:mainfrom
Autonymic:fix/clean-users

Conversation

@Autonymic
Copy link
Copy Markdown
Contributor

A minor fix that should close #2171.

Copy link
Copy Markdown
Contributor

@wookie184 wookie184 left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks!

@Xithrius Xithrius added a: backend Related to internal functionality and utilities (error_handler, logging, security, utils and core) a: moderation Related to community moderation functionality: (moderation, defcon, verification) p: 3 - low Low Priority t: enhancement Changes or improvements to existing features s: needs review Author is waiting for someone to review and approve labels May 27, 2022
@Xithrius Xithrius requested review from MrHemlock and ichard26 and removed request for Den4200, jb3 and ks129 May 27, 2022 22:10
Copy link
Copy Markdown
Contributor

@ichard26 ichard26 left a comment

Choose a reason for hiding this comment

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

Looks and works great to me, thank you @Autonymic! Good luck with your devops course!

@Xithrius Xithrius enabled auto-merge (squash) May 27, 2022 22:41
@Xithrius Xithrius merged commit 777e7e7 into python-discord:main May 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: backend Related to internal functionality and utilities (error_handler, logging, security, utils and core) a: moderation Related to community moderation functionality: (moderation, defcon, verification) p: 3 - low Low Priority s: needs review Author is waiting for someone to review and approve t: enhancement Changes or improvements to existing features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enable !clean users to take multiple users as arguments

4 participants