Skip to content

fix: add nvidia global theming#995

Merged
debermudez merged 2 commits into
mainfrom
dbermudez/aip-914-update-fern-theming
May 26, 2026
Merged

fix: add nvidia global theming#995
debermudez merged 2 commits into
mainfrom
dbermudez/aip-914-update-fern-theming

Conversation

@debermudez

@debermudez debermudez commented May 26, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • New Features

    • Added local documentation preview capability for staging and testing documentation changes.
  • Chores

    • Updated documentation framework to the latest version.
    • Applied NVIDIA theme styling to documentation.
    • Excluded local preview artifacts from version control to avoid committing generated preview files.

Review Change Stack

@github-actions

github-actions Bot commented May 26, 2026

Copy link
Copy Markdown

Try out this PR

Quick install:

pip install --upgrade --force-reinstall git+https://github.com/ai-dynamo/aiperf.git@3fdbadb792cb3f4e0bb4fac4bc05ce82feda3912

Recommended with virtual environment (using uv):

uv venv --python 3.12 && source .venv/bin/activate
uv pip install --upgrade --force-reinstall git+https://github.com/ai-dynamo/aiperf.git@3fdbadb792cb3f4e0bb4fac4bc05ce82feda3912

Last updated for commit: 3fdbadbBrowse code

@github-actions github-actions Bot added the fix label May 26, 2026
@debermudez debermudez force-pushed the dbermudez/aip-914-update-fern-theming branch from 236d3eb to 3fdbadb Compare May 26, 2026 21:45
@github-actions

github-actions Bot commented May 26, 2026

Copy link
Copy Markdown
@coderabbitai

coderabbitai Bot commented May 26, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 1ca13c8c-cc91-4522-95c3-256ef1647f57

📥 Commits

Reviewing files that changed from the base of the PR and between 236d3eb and 3fdbadb.

📒 Files selected for processing (4)
  • Makefile
  • fern/.gitignore
  • fern/docs.yml
  • fern/fern.config.json
✅ Files skipped from review due to trivial changes (2)
  • fern/fern.config.json
  • fern/.gitignore

Walkthrough

This PR adds a local Fern documentation preview workflow. A new fern-preview Make target stages documentation sources, converts Markdown to Fern MDX format, and runs the Fern dev server locally. Supporting changes update the Fern configuration version, apply the NVIDIA theme, and exclude the temporary staging directory from version control.

Changes

Fern Local Documentation Preview

Layer / File(s) Summary
Fern preview Make target implementation
Makefile
fern-preview target declared as .PHONY and implemented to validate the fern CLI, stage fern/ and docs/ sources into .local-preview/, run the MD→MDX conversion script, and launch fern docs dev locally.
Fern infrastructure configuration
fern/fern.config.json, fern/docs.yml, fern/.gitignore
Fern version updated to 5.37.10, NVIDIA global theme applied to documentation, and .local-preview/ directory excluded from version control.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A preview stage magically builds,
Staging docs with fern's gentle hand,
Theme of NVIDIA, our template now fills,
.local-preview hops out of sight, grand! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix: add nvidia global theming' directly reflects the main change: setting global-theme to nvidia in fern/docs.yml, making it clear and specific.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov

codecov Bot commented May 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@debermudez debermudez merged commit 03167d0 into main May 26, 2026
32 checks passed
@debermudez debermudez deleted the dbermudez/aip-914-update-fern-theming branch May 26, 2026 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 participants