Skip to content

DOC-7128 iris-radar-ri: convert iris/, radar/, and redisinsight/ images - #4188

Merged
andy-stark-redis merged 1 commit into
mainfrom
DOC-7128-iris-radar-ri
Oct 2, 2026
Merged

andy-stark-redis merged 1 commit into
mainfrom
DOC-7128-iris-radar-ri

Conversation

@andy-stark-redis

@andy-stark-redis andy-stark-redis commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Converts the image shortcodes in content/operate/iris/, content/operate/radar/, and content/operate/redisinsight/ to Markdown images, using the codemod from #4151. Part of DOC-7128.

Counts: all 86 images converted in 15 files (64 in Iris, 20 in Radar, 2 in Redis Insight). None needed skipping, so these trees have no image shortcodes left. 84 bare paths (images/...) were rewritten to site-root /images/.... There are no raw <img> tags in these trees. Radar's AGENTS.md has no image or link rules this conflicts with.

Rendered changes: none, apart from the dropped <a sdata-lightbox> wrapper.

Verification

  • Built main and this branch under the production baseURL (/docs/latest/): both succeeded (19,664 pages), and the hook reported no missing images.
  • Compared src/width/alt/class for every image on every page: no differences.
  • Coverage: the rendered sdata-lightbox wrappers on main match the source shortcodes exactly (64, 20, and 2), so all 86 are on published pages and the comparison saw every one. On this branch they're all gone.
  • The codemod deleted exactly one line per converted shortcode (86) and touched no file outside the list.
  • AI Markdown feed: the same 86 absolute image URLs before and after, with no attribute lines and no relative /images/ links.

🤖 Generated with Claude Code


Note

Low Risk
Documentation-only markup migration with no application or security logic changes; visual parity was verified in the PR except for dropped lightbox wrappers.

Overview
Replaces 86 Hugo image shortcodes in 15 operate docs pages (iris/, radar/, redisinsight/) with standard Markdown images and Hugo attribute blocks for width.

Each change follows the same pattern: {{<image filename="images/..." ...>}} becomes ![alt text](/images/...) on one line, with former width values moved to a following {width="..."} line where they existed. Image paths are normalized from images/... to site-root /images/.... Alt text is preserved from the shortcode.

Prose, tables, warnings, and other shortcodes are untouched. The only intentional behavioral change noted in the PR is removal of the rendered lightbox (sdata-lightbox) wrapper around screenshots.

Reviewed by Cursor Bugbot for commit 18b6cd4. Bugbot is set up for automated code reviews on this repo. Configure here.

Runs build/migrate_image_shortcodes.py over 15 files in content/operate/iris/,
radar/, and redisinsight/, converting all 86 image shortcodes to Markdown
images (84 bare paths rewritten to site-root /images/...). None needed
skipping, and these trees have no raw <img> tags.

Ticket: DOC-7128
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

DOC-7128

@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

🧠 Redis Memory

Found 5 related items from repository history (5 new this commit):

Memory updated at 18b6cd4

@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

@dwdougherty dwdougherty left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTC.

@andy-stark-redis
andy-stark-redis merged commit 9f20e31 into main Oct 2, 2026
100 checks passed
@andy-stark-redis
andy-stark-redis deleted the DOC-7128-iris-radar-ri branch October 2, 2026 13:53
EliShteinman added a commit to EliShteinman/docs that referenced this pull request Oct 3, 2026
Converts the Iris, Radar, and Redis Insight image shortcodes to Markdown images.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.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.

2 participants