Skip to content

Rename channels.discord_py to discord_bots#1982

Merged
Xithrius merged 4 commits into
mainfrom
bsoyka/discordpy-bots
Dec 15, 2021
Merged

Rename channels.discord_py to discord_bots#1982
Xithrius merged 4 commits into
mainfrom
bsoyka/discordpy-bots

Conversation

@bsoyka
Copy link
Copy Markdown
Contributor

@bsoyka bsoyka commented Nov 28, 2021

Since this is such a minor change, I'm just PRing it (at the advice of a core dev in #dev-contrib). However, there's a slight issue of whether this might affect stats too much, so I'm CCing @jb3 here for an opinion.

(Update: Joe believes this isn't a critical stat, so it should be fine to change.)

@bsoyka bsoyka added a: backend Related to internal functionality and utilities (error_handler, logging, security, utils and core) t: enhancement Changes or improvements to existing features labels Nov 28, 2021
@TizzySaurus TizzySaurus added the review: do not merge The PR can be reviewed but cannot be merged now label Nov 28, 2021
@TizzySaurus
Copy link
Copy Markdown
Contributor

Added do-not-merge label in case it messes with stats in a way we don't want.

Copy link
Copy Markdown
Contributor

@ToxicKidz ToxicKidz left a comment

Choose a reason for hiding this comment

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

Everythings looks good, except for one suggestion.

Comment thread bot/exts/moderation/slowmode.py Outdated
Co-authored-by: ToxicKidz <78174417+ToxicKidz@users.noreply.github.com>
@bsoyka bsoyka requested a review from ToxicKidz December 2, 2021 02:59
@mbaruh mbaruh removed the review: do not merge The PR can be reviewed but cannot be merged now label Dec 14, 2021
@Xithrius Xithrius added the p: 3 - low Low Priority label Dec 15, 2021
@Xithrius Xithrius enabled auto-merge (squash) December 15, 2021 14:01
@Xithrius Xithrius merged commit 40631e9 into main Dec 15, 2021
@Xithrius Xithrius deleted the bsoyka/discordpy-bots branch December 15, 2021 14:11
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) p: 3 - low Low Priority t: enhancement Changes or improvements to existing features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants