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

api: update return type of diff method #3115

Merged
merged 2 commits into from May 5, 2023
Merged

api: update return type of diff method #3115

merged 2 commits into from May 5, 2023

Conversation

john-b-yang
Copy link
Contributor

A minor fix, updating the documentation for the return type of the diff method for the Container model.

@milas
Copy link
Member

milas commented May 5, 2023

Thanks for the PR! Before I can accept this, I'll need you to make sure your commit has the Developer Certificate of Origin. See the instructions from CI @ https://github.com/docker/docker-py/pull/3115/checks?check_run_id=13126251061

@john-b-yang
Copy link
Contributor Author

@milas No problem and thanks for instructions! I believe I have made sure to include signoff on the editing commit. Happy to follow up on anything else that might be needed.

@milas milas merged commit 576e47a into docker:main May 5, 2023
10 checks passed
@milas
Copy link
Member

milas commented May 5, 2023

All set! Thanks for the PR 👍🏻

@john-b-yang
Copy link
Contributor Author

No problem! Thanks so much for maintaining the tool, it has been super helpful for some things I'm working on of late, really appreciate it 🙏🏼

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

Successfully merging this pull request may close these issues.

None yet

2 participants