Skip to content

test: manifest list backward compatibility #154

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
Merged
vbatts merged 1 commit into from
Jun 20, 2016
Merged

test: manifest list backward compatibility #154

vbatts merged 1 commit into from
Jun 20, 2016

Conversation

@runcom
Copy link
Member

@runcom runcom commented Jun 17, 2016 •
edited
Loading

Build on top of #151 and #152

commit 1d75bf1 adds manifest list test

@philips
Copy link
Contributor

philips commented Jun 17, 2016

need to fix the formatting on both commits :-/

@runcom
Copy link
Member Author

runcom commented Jun 17, 2016 •
edited
Loading

@philips fixed formatting in #152 and rebased this PR (#152 should go first - after #151, I'll rebase as needed to remove merged commits)

@runcom
test: exercise manifest list backwards compat
6f76343 
Signed-off-by: Antonio Murdaca <runcom@redhat.com>
@runcom
Copy link
Member Author

runcom commented Jun 17, 2016 •
edited
Loading

@philips actually, this fix can go in w/o any dependency on other PRs, PTAL

@philips
Copy link
Contributor

philips commented Jun 20, 2016 •
edited by caniszczyk
Loading

LGTM

Approved with PullApprove

1 similar comment
@vbatts
Copy link
Member

vbatts commented Jun 20, 2016 •
edited by caniszczyk
Loading

LGTM

Approved with PullApprove

@vbatts vbatts merged commit 82b42b6 into opencontainers:master Jun 20, 2016
@runcom runcom deleted the mlist branch June 20, 2016 23:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@Ghostman2591 Ghostman2591 Ghostman2591 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@runcom @philips @vbatts @Ghostman2591