Skip to content

fix: πŸ§‘β€πŸ’» Add kwargs explicitly for app-command decorators#3119

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

fix: πŸ§‘β€πŸ’» Add kwargs explicitly for app-command decorators#3119
ToothyDev wants to merge 10 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>
@Paillat-dev Paillat-dev requested a review from a team March 22, 2026 22:20
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.

Comments

@ToothyDev ToothyDev dismissed stale reviews from Lulalaby and Soheab via d11726f April 9, 2026 15:55
@ToothyDev ToothyDev requested a review from Paillat-dev April 9, 2026 15:55
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 9, 2026

This pull request does not follow the required pull request template.

Please use the default template (PULL_REQUEST_TEMPLATE.md) and fill out all required sections.

Problems detected:

The line "AI Usage has been disclosed." is missing.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 9, 2026

This pull request does not follow the required pull request template.

Please use the default template (PULL_REQUEST_TEMPLATE.md) and fill out all required sections.

Problems detected:

The line "AI Usage has been disclosed." is missing.

@Paillat-dev
Copy link
Copy Markdown
Member

Sorry for the trouble πŸ˜…

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