Skip to content

Conversation

BruhDark
Copy link
Contributor

@BruhDark BruhDark commented Oct 21, 2024

Summary

This PR adds with_counts for the fetch_guilds method, which was missing.

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.

@BruhDark
Copy link
Contributor Author

Side note 1: Blame my IDE for "fixing" spaces in other funcs in the files I edited.
Side note 2: Might wanna double check the edited documentation on such methods, don't know if it is correct.

@plun1331 plun1331 changed the title refactor: add missing with_counts parameter to fetch_guilds method feat: add missing with_counts parameter to fetch_guilds method Oct 21, 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.

Changelog please :3 (under Added)

@plun1331 plun1331 added priority: low Low Priority status: awaiting review Awaiting review from a maintainer feature Implements a feature API reflection Discord API isn't correctly reflected labels Oct 21, 2024
BruhDark and others added 3 commits October 21, 2024 11:01
Co-authored-by: Dorukyum <[email protected]>
Signed-off-by: Dark <[email protected]>
Co-authored-by: Dorukyum <[email protected]>
Signed-off-by: Dark <[email protected]>
@BruhDark
Copy link
Contributor Author

Done

plun1331
plun1331 previously approved these changes Oct 21, 2024
Co-authored-by: JustaSqu1d <[email protected]>
Signed-off-by: Dark <[email protected]>
auto-merge was automatically disabled October 25, 2024 22:15

Head branch was pushed to by a user without write access

Co-authored-by: JustaSqu1d <[email protected]>
Signed-off-by: Dark <[email protected]>
@BruhDark
Copy link
Contributor Author

Squid and his grammar.

@plun1331 plun1331 enabled auto-merge (squash) October 27, 2024 20:12
plun1331
plun1331 previously approved these changes Nov 3, 2024
@plun1331 plun1331 requested a review from JustaSqu1d November 3, 2024 15:52
@plun1331 plun1331 merged commit 48c5950 into Pycord-Development:master Nov 3, 2024
25 checks passed
OmLanke pushed a commit to OmLanke/pycord that referenced this pull request Dec 16, 2024
…d-Development#2615)

* refactor: Add missing `with_counts` parameter to guild retrieval methods

* Default with_counts to True in fetch_guilds

* Being dumb enough to forget about making it an int

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

* Update discord/client.py

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

* Update discord/iterators.py

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

* Update CHANGELOG.md

* Update discord/client.py

Co-authored-by: JustaSqu1d <[email protected]>
Signed-off-by: Dark <[email protected]>

* Update discord/iterators.py

Co-authored-by: JustaSqu1d <[email protected]>
Signed-off-by: Dark <[email protected]>

---------

Signed-off-by: Dark <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Dorukyum <[email protected]>
Co-authored-by: JustaSqu1d <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API reflection Discord API isn't correctly reflected feature Implements a feature priority: low Low Priority status: awaiting review Awaiting review from a maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants