Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

[Hold for next llm-d release] Add support for nested guide directories and clarify branch sync strategy#118

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
petecheslock wants to merge4 commits intollm-d:main
base:main
Choose a base branch
Loading
frompetecheslock:feat/support-remote-guides-in-dirs

Conversation

@petecheslock
Copy link
Collaborator

Changes

  1. Nested guide support - Added optionaltargetFilename parameter toDYNAMIC_GUIDES inguide-generator.js, allowing guides from nested directories (e.g.,prefix-cache-storage/cpu/README.md) to render as top-level sidebar entries instead of nested folders.

  2. Branch testing workflow - Updatedcomponents-data.yaml to support testing unreleased content by temporarily settingrelease.version to a feature branch name (currentlyliu-cong-debug for validation).

  3. Documentation improvements:

    • Added "Understanding Versioned vs. Always-Current Content" section explaining the dual sync strategy
    • Documented how versioned content (guides, architecture, components) pulls from release tags viaRELEASE_INFO.version
    • Documented how community docs always sync frommain branch viaCOMMON_REPO_CONFIGS
    • Added "Testing content from a feature branch" workflow
    • Added "Supporting remote guides from nested directories" guide withtargetFilename usage

Testing

Setrelease.version: liu-cong-debug incomponents-data.yaml and runnpm start to verify branch-based content sync works correctly.

Follow-up

Revertrelease.version back to the official release tag (e.g.,v0.3.1) before merging to production.

@petecheslockpetecheslock added the HOLDOn hold for _some_ reason. labelNov 17, 2025
@netlify
Copy link

netlifybot commentedNov 17, 2025•
edited
Loading

❌ Deploy Preview forelaborate-kangaroo-25e1ee failed.Why did it fail? →

NameLink
🔨 Latest commit7365c0b
🔍 Latest deploy loghttps://app.netlify.com/projects/elaborate-kangaroo-25e1ee/deploys/6920c928c4719b00084be859

@petecheslock
Copy link
CollaboratorAuthor

@liu-cong Check out this branch with your changes incorporated to add those 2 new guides. After those guides are merged and released we can then revert the test commit (e07366839c20217805c7a8994cd60335daa2d421) and then merge this with the release.

@petecheslockpetecheslockforce-pushed thefeat/support-remote-guides-in-dirs branch 2 times, most recently fromf794e65 to6af7508CompareNovember 20, 2025 19:20
@petecheslockpetecheslock marked this pull request as draftNovember 20, 2025 19:22
@petecheslockpetecheslock changed the title[DO NOT MERGE] Add support for nested guide directories and clarify branch sync strategy[Hold for next llm-d release] Add support for nested guide directories and clarify branch sync strategyNov 20, 2025
@petecheslock
Copy link
CollaboratorAuthor

Hold this until the new docs are included in the next llm-d release.

Signed-off-by: Pete Cheslock <pete.cheslock@redhat.com>
…dynamic guides- Updated README.md to include instructions for configuring remote guides from nested directories, detailing the use of `targetFilename` for top-level page generation.- Modified guide-generator.js to add new dynamic guides for 'Prefix Cache Storage' and 'Prefix Cache Storage - CPU', including sidebar positions and descriptions.Signed-off-by: Pete Cheslock <pete.cheslock@redhat.com>
Signed-off-by: Pete Cheslock <pete.cheslock@redhat.com>
@petecheslockpetecheslockforce-pushed thefeat/support-remote-guides-in-dirs branch from6af7508 to5353f0eCompareNovember 21, 2025 20:00
…tation- Renamed 'prefix-cache-storage' to 'tiered-prefix-cache' in guide-generator.js, updating titles and target filenames accordingly.- Adjusted .gitignore to reflect the new directory structure for tiered prefix cache documentation.Signed-off-by: Pete Cheslock <pete.cheslock@redhat.com>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

1 more reviewer

@liu-congliu-congliu-cong left review comments

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

@petecheslockpetecheslock

Labels

HOLDOn hold for _some_ reason.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@petecheslock@liu-cong
[8]ページ先頭
©2009-2025 Movatter.jp