Skip to content

Conversation

@qianzhangxa
Copy link
Contributor

No description provided.

@jonboulle
Copy link
Contributor

jonboulle commented Jan 20, 2017

LGTM, but can you squash the commits together?

Approved with PullApprove

@qianzhangxa
Copy link
Contributor Author

Thanks @jonboulle. But I am not sure how I can squash the commits together, should I do it here? Or in my forked repo?

@jonboulle
Copy link
Contributor

jonboulle commented Jan 20, 2017 via email

@qianzhangxa
Copy link
Contributor Author

Done.

@stevvooe
Copy link
Contributor

stevvooe commented Jan 20, 2017

LGTM

Approved with PullApprove

@stevvooe
Copy link
Contributor

Actually, needs a rebase.

@qianzhangxa
Copy link
Contributor Author

Rebased.

@jonboulle
Copy link
Contributor

jonboulle commented Jan 23, 2017

LGTM, again

Approved with PullApprove

@stevvooe
Copy link
Contributor

stevvooe commented Jan 24, 2017

LGTM

Approved with PullApprove

@stevvooe stevvooe merged commit fa40b3f into opencontainers:master Jan 24, 2017
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.

3 participants