Skip to content

Conversation

@yoggys
Copy link
Contributor

@yoggys yoggys commented May 17, 2024

Summary

Overall code type-hinting improvements.

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.

@yoggys
Copy link
Contributor Author

yoggys commented May 17, 2024

I think I caught most of what was wrong, this PR overwrites #2456 at the same time because I transferred the changes to this branch (so if it's ok then that one can be closed).

@yoggys yoggys marked this pull request as draft May 18, 2024 00:33
@plun1331 plun1331 added this to the v2.6 milestone May 18, 2024
@plun1331 plun1331 removed the request for review from CodeWithSwastik May 18, 2024 04:10
@plun1331 plun1331 added bug Something isn't working priority: low Low Priority status: in progress Work in Progess labels May 18, 2024
@yoggys
Copy link
Contributor Author

yoggys commented May 18, 2024

@plun1331 by now it should be all there, you can look at it at your free time

plun1331
plun1331 previously approved these changes May 28, 2024
Copy link
Member

@plun1331 plun1331 left a comment

Choose a reason for hiding this comment

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

I trust that you've double checked that these are all correct

Co-authored-by: Dorukyum <[email protected]>
Signed-off-by: YoggieS <[email protected]>
@Dorukyum Dorukyum enabled auto-merge (squash) June 5, 2024 20:48
@Dorukyum Dorukyum merged commit 61cc544 into Pycord-Development:master Jun 5, 2024
baribarton pushed a commit to baribarton/pycord-no-potential-reconnect that referenced this pull request Oct 24, 2024
* fix: RawMemberRemoveEvent type-hinting

* Update CHANGELOG.md

Signed-off-by: YoggieS <[email protected]>

* style(pre-commit): auto fixes from pre-commit.com hooks

* Update CHANGELOG.md

Co-authored-by: plun1331 <[email protected]>
Signed-off-by: YoggieS <[email protected]>

* fix: overall type-hinting

* style(pre-commit): auto fixes from pre-commit.com hooks

* fix: further types lookup & fixes

* style(pre-commit): auto fixes from pre-commit.com hooks

* Update discord/guild.py

Co-authored-by: UK <[email protected]>
Signed-off-by: YoggieS <[email protected]>

* review suggestions

* review suggestions v2

* style(pre-commit): auto fixes from pre-commit.com hooks

* review suggestions v3

* update changelog with correct #

* nonce typing partial revert

* onboarding PromptOption types

* Update CHANGELOG.md

Co-authored-by: Dorukyum <[email protected]>
Signed-off-by: YoggieS <[email protected]>

---------

Signed-off-by: YoggieS <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: plun1331 <[email protected]>
Co-authored-by: UK <[email protected]>
Co-authored-by: Dorukyum <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working priority: low Low Priority status: in progress Work in Progess

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants