Skip to content

DOC-7047: Migrate develop/clients/php to render hooks - #3946

Merged
andy-stark-redis merged 1 commit into
mainfrom
DOC-7047-clients-php
Sep 9, 2026
Merged

andy-stark-redis merged 1 commit into
mainfrom
DOC-7047-clients-php

Conversation

@andy-stark-redis

@andy-stark-redis andy-stark-redis commented Sep 8, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Converts content/develop/clients/php/ (8 files: _index.md, connect.md, error-handling.md, prob.md, queryjson.md, transpipe.md, vecsearch.md, vecsets.md) per DOC-7047, continuing the DOC-6909 render-hook rollout.
  • {{< relref ... >}} shortcode links (84 occurrences) rewritten to plain Markdown links in the canonical /content/<path>.md / /content/<path>/_index.md form, anchors preserved.
  • 3 {{< note >}}...{{< /note >}} callouts (no nesting, no custom titles, no warning/tip/info/alert usage in this directory) converted to > [!NOTE]-style blockquotes.
  • Uses the migration tooling from PR DOC-7047: Formalize DOC-6909 shortcode migration tooling #3937 (build/migrate_shortcode_links.py, not merged into this PR — pulled in as an untracked helper via git show, run, then removed before committing).

Test plan

  • python3 .claude/hooks/check_shortcode_paths.py --scan content/develop/clients/php/*.md reports 0 files with issues (0 broken relrefs, 0 broken file refs).
  • Manually reviewed the full diff: no remaining {{< relref or {{< note/warning/tip/info/alert }} shortcodes, no nested block-level shortcodes mangled inside a blockquote, blockquote formatting matches the redis-py reference conversion.
  • Full-site Hugo build verification (rendered-href diff) is being run centrally by the ticket owner before merge — this isolated worktree can't run the full Hugo build (gitignored generated deps like data/examples.json aren't present here).

Note

Low Risk
Documentation-only link and callout rewrites with no application or build-config changes; main risk is broken cross-links if canonical /content/... paths are wrong.

Overview
Continues the DOC-6909 render-hook rollout for DOC-7047 by updating all eight pages under content/develop/clients/php/.

{{< relref ... >}} links (84) are replaced with plain Markdown URLs in the canonical /content/<path>.md or /content/<path>/_index.md form, including fragment anchors. Three {{< note >}} callouts become GitHub-style > [!NOTE] blockquotes (Predis third-party disclaimer on the landing page, plus Predis v3 query dialect notes in queryjson.md and vecsearch.md).

Prose and {{< clients-example >}} snippets are unchanged; this is a linking and callout-format migration only.

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

Continues the DOC-6909 rollout to php: relref -> plain link -> canonical
/content/*.md form, and its 3 note callouts -> `> [!NOTE]`-style
blockquotes.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Sep 8, 2026 •

Copy link
Copy Markdown
Contributor

DOC-7047

@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

🧠 Redis Memory

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

Memory updated at ebb77ce

@andy-stark-redis

Copy link
Copy Markdown
Contributor Author

Central build verification (DOC-7047): built this branch merged onto main alongside the rest of batch 2, diffed rendered hrefs against a clean main build with build/diff_rendered_hrefs.py — 0 href differences, 0 new build warnings (17 pre-existing warnings identical on both sides, all unrelated to this change).

@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.

LGTM.

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