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.

Sign up for GitHub

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

Jump to bottom

Expose container properties of all images #590

Merged
merged 1 commit into from
Oct 23, 2016
Merged

Expose container properties of all images #590

merged 1 commit into from
Oct 23, 2016

Conversation

jpraet
Copy link
Contributor

@jpraet jpraet commented Oct 16, 2016 •
edited
Loading

fix for #583

@rhuss
Copy link
Collaborator

rhuss commented Oct 19, 2016

Thanks a lot. I'm currently on the road but will be back soon. Will check out the PR ASAP.

rhuss added a commit that referenced this pull request Oct 22, 2016
@rhuss
#583 : Refactored fix of #590 a bit to make the flow more clear
96723a2
@rhuss
Copy link
Collaborator

rhuss commented Oct 22, 2016

Thanks a lot ! I merged in the fix to branch integration (+ extracted some methods to make the flow of the function clearer).

will be available in the next release.

@rhuss rhuss added the applied label Oct 22, 2016
rhuss added a commit that referenced this pull request Oct 22, 2016
@rhuss
#583 : Refactored fix of #590 a bit to make the flow more clear
e46108d
@rhuss rhuss merged commit 1cbafa4 into fabric8io:master Oct 23, 2016
@rhuss rhuss mentioned this pull request Nov 11, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
applied
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jpraet @rhuss