Skip to content

added docstring for API class #86

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 1 commit into
base: trunk
Choose a base branch
from
Open

added docstring for API class #86

wants to merge 1 commit into from

Conversation

hemangjoshi37a
Copy link

added docstring for API class

@hemangjoshi37a hemangjoshi37a mentioned this pull request Feb 10, 2023
Open
@hemangjoshi37a
Copy link
Author

mentioning this in #88

aapjeisbaas
aapjeisbaas approved these changes Mar 31, 2023
View reviewed changes
Copy link

@aapjeisbaas aapjeisbaas left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Seems like a great explanation of what you can use the class for.
Hopefully someone picks up this merge.

@hemangjoshi37a
Copy link
Author

@aapjeisbaas thanks brother. that's so nice of you.

MilmanRonV
MilmanRonV approved these changes Aug 19, 2023
View reviewed changes
Copy link

@MilmanRonV MilmanRonV left a comment

Choose a reason for hiding this comment

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

The added docstring provides a clear overview of the API class and its methods. Nicely done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
2 more reviewers

@aapjeisbaas aapjeisbaas aapjeisbaas approved these changes

@MilmanRonV MilmanRonV MilmanRonV approved these changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hemangjoshi37a @aapjeisbaas @MilmanRonV