Skip to content
This repository has been archived by the owner on Aug 16, 2022. It is now read-only.

docs: update the warning in docker/README.md #122

Merged
merged 1 commit into from Apr 14, 2022

Conversation

kenji-miyake
Copy link
Contributor

@kenji-miyake kenji-miyake commented Mar 26, 2022

Description

Updated the warning to be the same as autowarefoundation/autoware-documentation#65.

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@kenji-miyake kenji-miyake requested a review from xmfcx March 26, 2022 08:37
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
@kenji-miyake
Copy link
Contributor Author

As @xmfcx pointed out in autowarefoundation/autoware#141 (comment), https://developer.nvidia.com/ngc/nvidia-deep-learning-container-license is broken.

To address this issue, I'll leave this PR as it is.
cc @esteve

@kenji-miyake
Copy link
Contributor Author

It seems the link was fixed.

@kenji-miyake kenji-miyake marked this pull request as ready for review April 14, 2022 13:48
@kenji-miyake kenji-miyake merged commit 8b4ed69 into main Apr 14, 2022
@kenji-miyake kenji-miyake deleted the update-docker-warning branch April 14, 2022 13:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants