Skip to content

fix: 🐛 Fix TypeError when typing ui.Select#2943

Merged
Paillat-dev merged 6 commits into
masterfrom
fix/type-error-select
Oct 8, 2025
Merged

fix: 🐛 Fix TypeError when typing ui.Select#2943
Paillat-dev merged 6 commits into
masterfrom
fix/type-error-select

Conversation

@Paillat-dev

Copy link
Copy Markdown
Member

Summary

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.

@pycord-app

pycord-app Bot commented Oct 2, 2025

Copy link
Copy Markdown

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/2943/head:pr-2943
git checkout pr-2943

This pull request can be installed with:

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

@Paillat-dev
Paillat-dev force-pushed the fix/type-error-select branch from b68ce72 to bddb527 Compare October 2, 2025 08:11

@Lumabots Lumabots left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm as long as its ok to bump the requierment

@Lumabots

Lumabots commented Oct 2, 2025

Copy link
Copy Markdown
Contributor

could we merge this asap ? as it fix a breaking change ? (even tho its still breaking in a way with the changement of the requirement

@Paillat-dev

Copy link
Copy Markdown
Member Author

The requirement isn't breaking because currently py-cord sub-demendencies require a higher typing_extensions version afaik

@Paillat-dev
Paillat-dev requested review from DA-344 and removed request for Icebluewolf and plun1331 October 8, 2025 15:30
@Paillat-dev
Paillat-dev merged commit 1d67b64 into master Oct 8, 2025
28 checks passed
@Paillat-dev
Paillat-dev deleted the fix/type-error-select branch October 8, 2025 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants