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

docs(podman_container): improve comments on network property #878

Merged
merged 1 commit into from
Nov 24, 2024

Conversation

vkhashimoto
Copy link
Contributor

closes #877

@vkhashimoto vkhashimoto force-pushed the docs/container-network branch 3 times, most recently from 186b13c to 5789a02 Compare November 24, 2024 03:19
@vkhashimoto
Copy link
Contributor Author

Oops, fixed the tests :)

plugins/modules/podman_container.py Outdated Show resolved Hide resolved
plugins/modules/podman_container.py Outdated Show resolved Hide resolved
plugins/module_utils/podman/podman_container_lib.py Outdated Show resolved Hide resolved
@vkhashimoto vkhashimoto force-pushed the docs/container-network branch 4 times, most recently from fbe4552 to fc03f80 Compare November 24, 2024 17:10
@vkhashimoto vkhashimoto force-pushed the docs/container-network branch from fc03f80 to 1ae8a1c Compare November 24, 2024 17:13
@vkhashimoto
Copy link
Contributor Author

@sshnaidm I've addressed all your comments and I'm only changing the description itself.

@sshnaidm
Copy link
Member

Thanks, looks much better!

@sshnaidm sshnaidm merged commit d68ca75 into containers:main Nov 24, 2024
18 checks passed
@vkhashimoto vkhashimoto deleted the docs/container-network branch November 24, 2024 20:40
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.

Documentation improvement on network property for podman_container
2 participants