Skip to content

Vendor in runtime spec referencing windows namespace#2644

Merged
dmcgowan merged 1 commit intocontainerd:masterfrom
madhanrm:vendor_win
Sep 15, 2018
Merged

Vendor in runtime spec referencing windows namespace#2644
dmcgowan merged 1 commit intocontainerd:masterfrom
madhanrm:vendor_win

Conversation

@madhanrm
Copy link
Copy Markdown
Contributor

Signed-off-by: Madhan Raj Mookkandy [email protected]

@madhanrm
Copy link
Copy Markdown
Contributor Author

@madhanrm
Copy link
Copy Markdown
Contributor Author

@jhowardmsft

Copy link
Copy Markdown

@lowenna lowenna left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread vendor.conf Outdated
@dmcgowan
Copy link
Copy Markdown
Member

The DCO seems to require a blank space between the subject and sign off. We can look into fixing this but when you push up the other change do you mind adding a blank line in your commit message after the subject to make the message validator happy.

@madhanrm
Copy link
Copy Markdown
Contributor Author

ping @dmcgowan can you approve this PR?

@madhanrm
Copy link
Copy Markdown
Contributor Author

@crosbymichael

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

Copy link
Copy Markdown
Member

@dmcgowan dmcgowan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dmcgowan dmcgowan merged commit 0dc7636 into containerd:master Sep 15, 2018
@madhanrm
Copy link
Copy Markdown
Contributor Author

madhanrm commented Sep 15, 2018

Thanks @dmcgowan @jhowardmsft @crosbymichael

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.

4 participants