Skip to content

fix: 🧑‍💻 Add kwargs explicitly for app-command decorators#3119

Open
ToothyDev wants to merge 12 commits intoPycord-Development:masterfrom
ToothyDev:fix/command-decorator-typing
Open

fix: 🧑‍💻 Add kwargs explicitly for app-command decorators#3119
ToothyDev wants to merge 12 commits intoPycord-Development:masterfrom
ToothyDev:fix/command-decorator-typing

Conversation

@ToothyDev
Copy link
Copy Markdown
Contributor

@ToothyDev ToothyDev commented Feb 21, 2026

Summary

This explicitly adds typed kwargs to the slash_command, user_command and message_command decorators as well as to the basic application_command decorator.

Information

  • This PR fixes an issue.
  • This PR adds something new (e.g. new method or parameters).
  • This PR is a breaking change (e.g. methods or parameters removed/renamed).
  • This PR is not a code change (e.g. documentation, README, typehinting,
    examples, ...).

Checklist

  • I have searched the open pull requests for duplicates.
  • If code changes were made then they have been tested.
    • I have updated the documentation to reflect the changes.
  • If type: ignore comments were used, a comment is also left explaining why.
  • I have updated the changelog to include these changes.
  • AI Usage has been disclosed.
    • If AI has been used, I understand fully what the code does

@pycord-app
Copy link
Copy Markdown

pycord-app bot commented Feb 21, 2026

Thanks for opening this pull request!
Please make sure you have read the Contributing Guidelines and Code of Conduct.

This pull request can be checked-out with:

git fetch origin pull/3119/head:pr-3119
git checkout pr-3119

This pull request can be installed with:

pip install git+https://github.com/Pycord-Development/pycord@refs/pull/3119/head

Paillat-dev
Paillat-dev previously approved these changes Feb 28, 2026
Copy link
Copy Markdown
Member

@Paillat-dev Paillat-dev left a comment

Choose a reason for hiding this comment

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

Should do the trick for now

Paillat-dev
Paillat-dev previously approved these changes Feb 28, 2026
Copy link
Copy Markdown
Member

@Paillat-dev Paillat-dev left a comment

Choose a reason for hiding this comment

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

Should do the trick for now

Signed-off-by: ToothyDev <55001472+ToothyDev@users.noreply.github.com>
@github-actions

This comment was marked as resolved.

@Paillat-dev
Copy link
Copy Markdown
Member

Sorry for the trouble 😅

@Paillat-dev Paillat-dev removed the invalid This doesn't seem right label Apr 11, 2026
Copy link
Copy Markdown
Member

@Paillat-dev Paillat-dev left a comment

Choose a reason for hiding this comment

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

See my other comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

8 participants