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

Update nvidia-container-toolkit.rst #18

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

pozsa
Copy link

@pozsa pozsa commented Oct 26, 2020

This piece of information feels important to be included in the docs.

#17

This piece of information feels important to be included in the docs.

eth-cscs#17
@pozsa
Copy link
Author

pozsa commented Oct 27, 2020

@Madeeks the failing job step looks to me unrelated to this PR's content.

@Madeeks
Copy link
Member

Madeeks commented Nov 4, 2020

Hello @pozsa, thank you for opening a PR!
In order to proceed I have to ask you two things:

  1. Could you please point the PR to merge into the develop branch instead of master?
  2. To submit a contribution it is necessary that you sign a Contributor's License Agreement (CLA) with ETH Zurich. I see that you have no public email address in your profile, so could you please send me an email with your contact so that I can provide you the CLA form? You may find my address in my GitHub profile. Thanks!

@pozsa pozsa changed the base branch from master to develop November 10, 2020 14:38
@pozsa
Copy link
Author

pozsa commented Nov 16, 2020

@Madeeks I've sent you an email. Did you receive it?

@Madeeks
Copy link
Member

Madeeks commented Dec 9, 2020

Hi @pozsa, I received your mail and sent the CLA to the corresponding address some time ago. Let me know if you didn't receive it and I should send it again.

@Madeeks Madeeks added the enhancement New feature or request label Feb 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants