Skip to content
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

Docstrings for the function copy_object #1071

Merged
merged 6 commits into from
May 25, 2023

Conversation

linwaytin
Copy link
Contributor

Add docstrings for the copy_object methods.

@mkitti
Copy link
Member

mkitti commented May 20, 2023

Great job so far.

I think we will need a new file docs.

@linwaytin
Copy link
Contributor Author

Didn't know the docs part. Thank you!

@mkitti
Copy link
Member

mkitti commented May 20, 2023

Did you want to document the other function you were interested in as well?

@linwaytin
Copy link
Contributor Author

Yes! Let me do that for delete_object.

@mkitti
Copy link
Member

mkitti commented May 20, 2023

Let me know when you are done.

@linwaytin
Copy link
Contributor Author

I added the docstring for delete_object, and a short description for copy_object so that people know its existence.

If everything is OK, we can merge it.

@mkitti mkitti merged commit 9bd0032 into JuliaIO:master May 25, 2023
14 of 15 checks passed
@linwaytin
Copy link
Contributor Author

Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants