-
-
Notifications
You must be signed in to change notification settings - Fork 110
Conversation
Codecov Report
@@ Coverage Diff @@
## main #174 +/- ##
=======================================
Coverage 73.84% 73.84%
=======================================
Files 21 21
Lines 906 906
Branches 221 227 +6
=======================================
Hits 669 669
Misses 235 235
Partials 2 2
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe add some check mechanism instead of setting guild commands again and again.
const app = await client.application?.fetch(); | ||
const devGuild = await client.guilds.fetch(deployment_guild_id).catch(() => null); | ||
if (app && devGuild) { | ||
await devGuild.commands.set([ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be great to have some check if the commands are already present in the guild before setting commands again and again at every start of bot.
@Fyko could you remove the unnecessary intent mentioned above and also move command deployment to its own script that can be run separately? |
Co-authored-by: DaStormer <40336269+DaStormer@users.noreply.github.com>
Why it's not getting merged? |
Please describe the changes this PR makes and why it should be merged:
This PR updates examples for Discord.js v13. Additionally, it fixes a documentation error.
Closes #170.
Status and versioning classification: