Skip to content

fix example yaml - #709

Merged
archlitchi merged 3 commits into
Project-HAMi:masterfrom
lixd:fix
Dec 18, 2024
Merged

fix example yaml#709
archlitchi merged 3 commits into
Project-HAMi:masterfrom
lixd:fix

Conversation

@lixd

@lixd lixd commented Dec 16, 2024

Copy link
Copy Markdown
Contributor

add detail comment for nvidia specify card case.

What type of PR is this?

/kind documentation
What this PR does / why we need it:
add detail comment to nvidia specify card case to be more user-friendly. avoid wrong use,like #707
Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

add detail comment for nvidia specify card case.

Signed-off-by: lixd <xueduan.li@gmail.com>
Comment thread examples/nvidia/specify_card_type_not_use.yaml Outdated
Signed-off-by: lixd <xueduan.li@gmail.com>
Comment thread examples/nvidia/specify_uuid_to_use.yaml Outdated
Signed-off-by: lixd <xueduan.li@gmail.com>
@lixd

lixd commented Dec 18, 2024

Copy link
Copy Markdown
Contributor Author

@archlitchi have a look

@archlitchi

Copy link
Copy Markdown
Member

/lgtm

@archlitchi
archlitchi merged commit 4785c8f into Project-HAMi:master Dec 18, 2024
@codecov

codecov Bot commented Dec 18, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Flag Coverage Δ
unittests 37.03% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@lixd
lixd deleted the fix branch January 14, 2025 00:43
@fishman fishman mentioned this pull request Jul 1, 2026
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