-
Notifications
You must be signed in to change notification settings - Fork 401
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.
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
Addition of docstrings to the files #61
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, thanks for taking the time to send a PR!
This mostly looks good, I just fixed some formatting in the first file, could you fix the couple of comments in the other two?
@@ -63,6 +91,21 @@ def _next_page(self): | |||
|
|||
|
|||
def _list_models(page_size, page_token, client): | |||
""" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here too
@MarkDaoust, Sure I will update it as per your guidance and |
6d057a7
to
181c956
Compare
181c956
to
627e851
Compare
That should work this time. |
Thanks! |
* update model.py * more * Add back-ticks, remove some redundant types. * update model.py 1 * fomatting * update text.py * Formatting fixes --------- Co-authored-by: Mark Daoust <[email protected]>
close #60