DOC-7104: Migrate content/operate/rs/flex/, monitoring/, and networking/ to render hooks - #4092
Open
andy-stark-redis wants to merge 1 commit into
Open
andy-stark-redis wants to merge 1 commit into
andy-stark-redis wants to merge 1 commit into
Conversation
…ng/ to render hooks Converts relref link shortcodes to plain Markdown links and note/tip/warning/alert callout shortcodes to `> [!NOTE]`-style blockquotes across all 24 files in content/operate/rs/flex/ (4), content/operate/rs/monitoring/ (12), and content/operate/rs/networking/ (8) — resolved by render-link.html and render-blockquote.html per the DOC-6909 render-hook migration. Purely mechanical; no prose, terminology, or facts changed. This is unit 7 of 15 and the last unit for the live/unversioned tree. 132 relref shortcodes and 17 callout shortcodes converted (measured directly by grep, not from the ticket's rough estimate). 22 of 24 files changed; flex/scale.md's sibling observability.md and networking/_index.md had none to convert. Hand-fixed 4 instances of the known list-indented-callout gotcha (the converter only preserves indentation on a blockquote's header line, leaving continuation lines unindented and a stray whitespace-only `>` line): flex/get-started.md, monitoring/get-started.md (x2), monitoring/metrics_stream_engine/_index.md. Verified via rendered HTML that <li> nesting is unchanged before/after in each case. No instances found of the other two known gotchas (relref with no separating slash before trailing path text; relref missing a paren). diff_rendered_hrefs.py (patched version from PR #4087, not part of this commit) reports 0 href diffs across all three subdirectory prefixes. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Contributor
Contributor
Contributor
🧠 Redis MemoryFound 5 related items from repository history (5 new this commit):
Memory updated at 30e0ef6 |
6 tasks
andy-stark-redis
requested review from
dwdougherty
and removed request for
kaitlynmichael
September 23, 2026 15:20
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Unit 7 of 15 in the shortcode-to-render-hook migration (DOC-6909/DOC-7104) — and the last unit for the live/unversioned tree (units 8–15 cover frozen version snapshots).
Converts every file under
content/operate/rs/flex/(4 files),content/operate/rs/monitoring/(12 files), andcontent/operate/rs/networking/(8 files) — 24 files total, 22 of which had a shortcode to convert:relreflink shortcodes → plain Markdown links (/content/<path>.md[#anchor]), resolved bylayouts/_default/_markup/render-link.htmlnote/tip/warning/alertcallout shortcodes →> [!NOTE]-style blockquotes, resolved bylayouts/_default/_markup/render-blockquote.htmlPurely mechanical syntax migration. No prose, terminology, or facts were touched.
Real counts (measured directly by grep before converting, not from the ticket's rough estimate): 132 relref shortcodes and 17 callout shortcodes. These matched the ticket's rough estimate (~130 relref, ~17 callouts) for this particular unit.
Gotcha instances found
content/operate/rs/flex/get-started.md,content/operate/rs/monitoring/get-started.md(×2),content/operate/rs/monitoring/metrics_stream_engine/_index.md. The converter only preserves indentation on a blockquote's header line, leaving continuation/closing lines unindented and a stray whitespace-only>artifact line, which detaches the blockquote from its<li>nesting. Hand-fixed by re-indenting continuation lines and removing the stray line. Verified via rendered HTML that<li>nesting is unchanged before/after in each case.Verification
hugo --minify).build/diff_rendered_hrefs.py(from PR DOC-7104: Fix diff_rendered_hrefs.py to catch unquoted href attributes #4087, fixing a regex that only matched double-quoted href attributes) for each of the three subdirectory prefixes: 0 href diffs inoperate/rs/flex,operate/rs/monitoring, andoperate/rs/networking.build/diff_rendered_hrefs.pyitself is not part of this diff (git diff --stat origin/main -- build/is empty).Test plan
content/operate/rs/flex/,content/operate/rs/monitoring/, andcontent/operate/rs/networking/on preview🤖 Generated with Claude Code
Note
Low Risk
Documentation-only syntax migration with no product or runtime changes; risk is limited to broken links or callout rendering on preview.
Overview
This PR continues the shortcode-to-render-hook migration (DOC-7104) for 24 live docs under
content/operate/rs/flex/,monitoring/, andnetworking/.Internal links: Hugo
{{< relref "..." >}}shortcodes are replaced with plain Markdown links using/content/...paths (and anchors where needed), whichrender-link.htmlresolves likerelref.Callouts:
{{< note >}},{{< warning >}}, and similar shortcodes become GitHub-style alert blockquotes (> [!NOTE],> [!WARNING], etc.), handled byrender-blockquote.html.A few list-nested callouts were hand-fixed so blockquotes stay indented under their list items. Prose and facts are unchanged; verification reported zero rendered href diffs for these trees.
Reviewed by Cursor Bugbot for commit 30e0ef6. Bugbot is set up for automated code reviews on this repo. Configure here.