Skip to content

docs(rocminfo, rocshmem): Update html theme options - #12908

Merged
yugang-amd merged 2 commits into
developfrom
rtd-theme-update
Oct 7, 2026
Merged

yugang-amd merged 2 commits into
developfrom
rtd-theme-update

Conversation

@yugang-amd

Copy link
Copy Markdown
Contributor

JIRA ID : AIROCDOC-4325

Update the HTML theme options to align with ROCm Core SDK docs.

Copilot AI balanced review requested due to automatic review settings October 6, 2026 13:48
@github-actions github-actions Bot added documentation Improvements or additions to documentation project: rocminfo project: rocshmem labels Oct 6, 2026
@therock-pr-bot

therock-pr-bot Bot commented Oct 6, 2026

Copy link
Copy Markdown

✅ All Policy Checks Passed

Check Status Details
📝 PR Description ✅ Pass —
⛔ Forbidden Files ✅ Pass —
🧪 Unit Test ⚠️ Warning Error: Source/code files changed without an accompanying unit test.
Expected: add at least one test file named like test_<name>.py / test_<name>.cpp (or <name>_test.*).
Current: code file(s) changed: projects/rocminfo/docs/conf.py, projects/rocshmem/docs/conf.py; no test file found
🚫 Draft PR 🔜 To Be Enabled —
🚩 Feature Flag 🔜 To Be Enabled —
📊 Code Coverage 🔜 To Be Enabled —

🎉 All policy checks passed!

📖 Need help? See the Policy FAQ for details on every check and how to fix failures.

🙋 Wish to Override Policy?

@yugang-amd

Copy link
Copy Markdown
Contributor Author

/docs-preview

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The rocSHMEM configuration incorrectly points documentation links to the rocdbgapi directory.

Review effort: Balanced
Findings: 1 Medium severity

Open (1)
What changed in this PR

Updates rocSHMEM and rocminfo documentation theme options to align with ROCm Core SDK documentation.

Changes:

  • Adds repository, issue, and download buttons.
  • Updates the rocSHMEM copyright year.
File Description
projects/​rocshmem/​docs/​conf.py Adds theme options and updates copyright.
projects/​rocminfo/​docs/​conf.py Adds repository-related theme options.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread projects/rocshmem/docs/conf.py Resolved
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@systems-assistant

Copy link
Copy Markdown
Contributor

Docs preview for rocminfo is building on Read the Docs: https://rocm.docs.amd.com/projects/rocminfo/en/rtd-theme-update/ (Note: If a pull request includes multiple projects, only the preview for the first project will be built.)

@yugang-amd
yugang-amd merged commit 0f0bb28 into develop Oct 7, 2026
47 of 49 checks passed
@yugang-amd
yugang-amd deleted the rtd-theme-update branch October 7, 2026 12:55
systems-assistant Bot pushed a commit to ROCm/rocminfo that referenced this pull request Oct 7, 2026
JIRA ID : AIROCDOC-4325

Update the HTML theme options to align with ROCm Core SDK docs.
[rocm-systems] ROCm/rocm-systems#12908 (commit 0f0bb28)
systems-assistant Bot pushed a commit to ROCm/rocSHMEM that referenced this pull request Oct 7, 2026
JIRA ID : AIROCDOC-4325

Update the HTML theme options to align with ROCm Core SDK docs.
[rocm-systems] ROCm/rocm-systems#12908 (commit 0f0bb28)
yugang-amd added a commit that referenced this pull request Oct 7, 2026
JIRA ID : AIROCDOC-4325

Update the HTML theme options to align with ROCm Core SDK docs.

---------

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@systems-assistant

Copy link
Copy Markdown
Contributor

TheRock Submodule Bump Activity

Newest first

anujshuk-amd pushed a commit that referenced this pull request Oct 12, 2026
JIRA ID : AIROCDOC-4325

Update the HTML theme options to align with ROCm Core SDK docs.

---------

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants