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

Contruct a container object by podman version #15

Merged
merged 1 commit into from
Sep 3, 2020
Merged

Conversation

rgolangh
Copy link
Owner

@rgolangh rgolangh commented Sep 3, 2020

This change will now build a container object and extract the info from
the json output by the podman version.

Now we can make the extension work with various podman version and break
less often by those changes.

Credit and many thanks to @rmarting 🙏 who sent this #14 which drove me to invest some time in making the code better suited for multi versions.

Signed-off-by: Roy Golan [email protected]

This change will now build a container object and extract the info from
the json output by the podman version.

Now we can make the extension work with various podman version and break
less often by those changes.

Signed-off-by: Roy Golan <[email protected]>
@rmarting
Copy link

rmarting commented Sep 3, 2020

My user is @rmarting 😄 Anyway it was a pleassure to help you.

@rgolangh
Copy link
Owner Author

rgolangh commented Sep 3, 2020

My user is @rmarting smile Anyway it was a pleassure to help you.

noted and fixed 🤦 🎊

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.

2 participants