Document that the dali-dynamic-mode skill exists#6358
Conversation
|
| Filename | Overview |
|---|---|
| docs/dali_dynamic/overview.rst | Adds a .. tip:: admonition documenting the dali-dynamic-mode skill with install instructions; RST syntax is correct and placement is logical. |
| README.rst | Inserts a GitHub-only [!TIP] pull-quote for the skill (hidden in Sphinx via injected CSS) and promotes "DALI in action" to a proper section heading; mechanism is sound. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[README.rst and overview.rst] --> B{Rendering context}
B -->|GitHub RST preview| C[pull-quote renders as blockquote - GH admonition visible]
B -->|Sphinx HTML| D[CSS injected - pull-quote hidden via github-only class]
B -->|Sphinx HTML| E[tip directive in overview.rst renders as native tip box]
Reviews (11): Last reviewed commit: "Fix broken links to skill" | Re-trigger Greptile
|
!build |
|
CI MESSAGE: [51951217]: BUILD STARTED |
|
CI MESSAGE: [51951217]: BUILD PASSED |
|
@rostan-t - how about adding this note also to the readme, maybe just above the examples and after the about section? |
6ef9c45 to
efbc486
Compare
|
!build |
|
CI MESSAGE: [52661253]: BUILD STARTED |
|
CI MESSAGE: [52661253]: BUILD PASSED |
Signed-off-by: Rostan Tabet <rtabet@nvidia.com>
Signed-off-by: Rostan Tabet <rtabet@nvidia.com>
Signed-off-by: Rostan Tabet <rtabet@nvidia.com>
|
!build |
|
CI MESSAGE: [53388427]: BUILD STARTED |
Signed-off-by: Rostan Tabet <rtabet@nvidia.com>
|
!build |
|
CI MESSAGE: [53389553]: BUILD STARTED |
|
CI MESSAGE: [53389553]: BUILD FAILED |
Category:
Other (e.g. Documentation, Tests, Configuration)
Description:
Point to the dali-dynamic-mode skill in NVIDIA/skills in the documentation.
Additional information:
Affected modules and functionalities:
Dynamic API Overview documentation page:

Key points relevant for the review:
Tests:
Checklist
Documentation
DALI team only
Requirements
REQ IDs: N/A
JIRA TASK: N/A