Skip to content

feat: adding missing parameters and attribute documentation #2772

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 49 commits into
base: master
Choose a base branch
from
Open

feat: adding missing parameters and attribute documentation #2772

wants to merge 49 commits into from

Conversation

Lumabots
Copy link
Contributor

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.

Lumabots and others added 17 commits March 29, 2025 14:08
@Lumabots
Update guild.py
426bb6a 
Signed-off-by: Lumabots <144063653+Lumabots@users.noreply.github.com>
@Lumabots
Update channel.py
6e68b7a 
add missings docs

Signed-off-by: Lumabots <144063653+Lumabots@users.noreply.github.com>
@Lumabots
Update guild.py
46aa608 
Signed-off-by: Lumabots <144063653+Lumabots@users.noreply.github.com>
@pre-commit-ci
style(pre-commit): auto fixes from pre-commit.com hooks
332e652
@Lumabots
Update channel.py
eff4e52 
Signed-off-by: Lumabots <144063653+Lumabots@users.noreply.github.com>
@pre-commit-ci
style(pre-commit): auto fixes from pre-commit.com hooks
477407f
@Lumabots
Update guild.py
e717f67 
Signed-off-by: Lumabots <144063653+Lumabots@users.noreply.github.com>
@Lumabots
Update guild.py
13c6037 
Signed-off-by: Lumabots <144063653+Lumabots@users.noreply.github.com>
@pre-commit-ci
style(pre-commit): auto fixes from pre-commit.com hooks
c4b7091
@Lumabots
Update CHANGELOG.md
1d7eb3f 
Signed-off-by: Lumabots <144063653+Lumabots@users.noreply.github.com>
@pre-commit-ci
style(pre-commit): auto fixes from pre-commit.com hooks
dd0f397
@Lumabots
Update CHANGELOG.md
340cec3 
Signed-off-by: Lumabots <144063653+Lumabots@users.noreply.github.com>
@pre-commit-ci
style(pre-commit): auto fixes from pre-commit.com hooks
bc2fef9
@Lumabots
Update guild.py
d8b79ff 
Signed-off-by: Lumabots <144063653+Lumabots@users.noreply.github.com>
@Lumabots
Update channel.py
952aab8 
removal of all the nsfw mention which should not exist for category

Signed-off-by: Lumabots <144063653+Lumabots@users.noreply.github.com>
@Lumabots
Update channel.py
7dd8a7a 
Signed-off-by: Lumabots <144063653+Lumabots@users.noreply.github.com>
@pre-commit-ci
style(pre-commit): auto fixes from pre-commit.com hooks
0035a73
@Lumabots Lumabots requested a review from a team as a code owner April 30, 2025 19:32
@pullapprove4 pullapprove4 bot requested a review from VincentRPS April 30, 2025 19:32
@Lumabots
Copy link
Contributor Author

sorry i renamed the branch and it deleted the other one

@Lumabots Lumabots changed the title Channel docs: Improve docstrings and attribute documentation Apr 30, 2025
@Lumabots
Merge branch 'master' into channel
1ef8618 
Signed-off-by: Lumouille <144063653+Lumabots@users.noreply.github.com>
@plun1331 plun1331 mentioned this pull request May 1, 2025
Closed
8 tasks
plun1331
plun1331 requested changes May 1, 2025
View reviewed changes
Lumabots and others added 2 commits May 1, 2025 19:45
@Lumabots @plun1331
fix typo
757ef9c 
Co-authored-by: plun1331 <plun1331@gmail.com>
Signed-off-by: Lumouille <144063653+Lumabots@users.noreply.github.com>
@Lumabots @plun1331
Update discord/channel.py
cab128e 
Co-authored-by: plun1331 <plun1331@gmail.com>
Signed-off-by: Lumouille <144063653+Lumabots@users.noreply.github.com>
@Lumabots
Copy link
Contributor Author

This pull request title is also misleading.

if you have a better name in mind i'll be happy to change it

JustaSqu1d
JustaSqu1d requested changes Jun 20, 2025
View reviewed changes
Copy link
Member

@JustaSqu1d JustaSqu1d left a comment

Choose a reason for hiding this comment

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

Fix markdown

@JustaSqu1d
Copy link
Member

This pull request title is also misleading.

if you have a better name in mind i'll be happy to change it

This pull request isn't a documentation change.

@Lumabots @JustaSqu1d
Update CHANGELOG.md
4b7ca07 
Co-authored-by: JustaSqu1d <89910983+JustaSqu1d@users.noreply.github.com>
Signed-off-by: Lumouille <144063653+Lumabots@users.noreply.github.com>
@Lumabots Lumabots changed the title docs: Improve docstrings and attribute documentation feats: adding missing parameters and attribute documentation Jun 20, 2025
Lumabots and others added 5 commits June 20, 2025 19:14
@Lumabots @JustaSqu1d
Update CHANGELOG.md
4d35fee 
Co-authored-by: JustaSqu1d <89910983+JustaSqu1d@users.noreply.github.com>
Signed-off-by: Lumouille <144063653+Lumabots@users.noreply.github.com>
@Lumabots @JustaSqu1d
Update CHANGELOG.md
d1f1602 
Co-authored-by: JustaSqu1d <89910983+JustaSqu1d@users.noreply.github.com>
Signed-off-by: Lumouille <144063653+Lumabots@users.noreply.github.com>
@Lumabots
Merge branch 'master' into channel
6c8b79e
@Lumabots
added reference
7970680
@Lumabots
Merge branch 'channel' of https://github.com/Lumabots/pycord into cha…
f073663 
…nnel
@Lumabots Lumabots changed the title feats: adding missing parameters and attribute documentation feat: adding missing parameters and attribute documentation Jun 20, 2025
Paillat-dev
Paillat-dev reviewed Jul 17, 2025
View reviewed changes
@Lumabots Lumabots requested a review from Paillat-dev July 22, 2025 19:23
Paillat-dev
Paillat-dev previously approved these changes Jul 22, 2025
View reviewed changes
Copy link
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.

lgtm

Lulalaby
Lulalaby requested changes Aug 2, 2025
View reviewed changes
Copy link
Member

@Lulalaby Lulalaby left a comment

Choose a reason for hiding this comment

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

Merge conflicts

@Lumabots
Merge branch 'master' into channel
e126d12 
Signed-off-by: Lumouille <144063653+Lumabots@users.noreply.github.com>
@Lumabots
Copy link
Contributor Author

Lumabots commented Aug 2, 2025

fixed

Lulalaby
Lulalaby previously approved these changes Aug 2, 2025
View reviewed changes
@Lumabots
Merge branch 'master' into channel
7c10a40 
Signed-off-by: Lumouille <144063653+Lumabots@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Paillat-dev Paillat-dev Paillat-dev left review comments

@JustaSqu1d JustaSqu1d JustaSqu1d requested changes

@Lulalaby Lulalaby Lulalaby left review comments

@VincentRPS VincentRPS Awaiting requested review from VincentRPS

@plun1331 plun1331 Awaiting requested review from plun1331

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
PA: Maintainers pending
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@Lumabots @JustaSqu1d @Lulalaby @plun1331 @Paillat-dev @Dorukyum