-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
feat: add applicationCommandMention()
formatter
#8280
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov Report
@@ Coverage Diff @@
## main #8280 +/- ##
==========================================
+ Coverage 86.40% 86.51% +0.11%
==========================================
Files 87 87
Lines 8221 8290 +69
Branches 1098 1102 +4
==========================================
+ Hits 7103 7172 +69
Misses 1075 1075
Partials 43 43
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. |
da45b86
to
dee1dde
Compare
062bccc
to
d3952c4
Compare
This PR needs a rebase. |
d3952c4
to
f779804
Compare
Rebased |
b8ccb44
to
d229f65
Compare
64e4fab
to
a59d422
Compare
You cannot include options in command mentions |
That's true, but now that there are attachment options, |
a59d422
to
00990c9
Compare
Can't reopen this pull request now. Made a new one, #8546 |
Please describe the changes this PR makes and why it should be merged:
applicationCommandMention()
formatted in /buildersApplicationCommand#toString()
, which will use the function mentioned above to return the commands mentionUpstream:
Status and versioning classification: