Skip to content

Redirect retired /guides tag pages to /dev/tutorials - #21397

Merged
cnunciato merged 1 commit into
masterfrom
cnunciato/guides-redirects-dev-tutorials
Sep 4, 2026
Merged

Redirect retired /guides tag pages to /dev/tutorials#21397
cnunciato merged 1 commit into
masterfrom
cnunciato/guides-redirects-dev-tutorials

Conversation

@cnunciato

@cnunciato cnunciato commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Adds 168 static redirects to scripts/redirects/dev-redirects.txt for retired /guides/* pages that weren't covered in #21095. This mainly includes taxonomy pages (e.g, /tags/*), which aren't normally indexed, but for some reason were in the Guides, so this picks up those pages and lands them at the Tutorials index so as not to drop them on the floor.

@github-actions github-actions Bot added review:triaging Claude Triage is currently classifying the PR domain:infra PR touches workflows, scripts, infra, Makefile, or build config labels Sep 4, 2026
@cnunciato
cnunciato requested a review from CamSoper September 4, 2026 03:43
@github-actions github-actions Bot removed the review:triaging Claude Triage is currently classifying the PR label Sep 4, 2026
@cnunciato
cnunciato requested a review from jkodroff September 4, 2026 03:43
@github-actions github-actions Bot added the review:in-progress Claude review is currently running label Sep 4, 2026
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Pre-merge Review — Last updated 2026-09-04T03:43:50Z

Tip

Summary: Infrastructure-only change: 168 new S3 redirect entries in scripts/redirects/dev-redirects.txt sending the retired /guides taxonomy tag pages (166 of them) plus the /guides/how-to/ and /guides/neo-prompts/ section indexes to /dev/tutorials/. The failure modes that would matter here are a malformed line (wrong separator, missing destination), a duplicate source path colliding with an existing rule, or a redirect pointed at a page that is still live — so the passes that ran were a format check on every added line, a whole-file duplicate-source scan, and a repo-wide search for the two non-tag paths. All 168 lines use the source|destination form with a /dev/tutorials/ target, there are no duplicate sources anywhere in the file, the entries sort in with the existing guides/… block and match the file's established style, and neither guides/how-to/ nor guides/neo-prompts/ is referenced by live content in this repo. No content files changed, so there were no claims to fact-check.

Review confidence:

Dimension Level Notes
mechanics HIGH
facts HIGH No prose or factual claims in this diff.
Investigation log
  • Cross-sibling reads: not run (not in a templated section)
  • External claim verification: not run (no claims in this diff)
  • Cited-claim spot-checks: not run (no cited claims)
  • Frontmatter sweep: not run (no frontmatter in diff)
  • Temporal-trigger sweep: not run (no trigger words)
  • Code execution: not run (no static/programs/ change)
  • Code-examples checks: not run (no fenced code blocks in content files)
  • Editorial-balance pass: not run (not under content/blog/)
🚨 Outstanding ⚠️ Low-confidence 💡 Pre-existing ✅ Resolved
0 0 0 0

🔍 Verification trail

No verifiable claims extracted from this diff.

🚨 Outstanding in this PR

No outstanding findings in this PR.

⚠️ Low-confidence

Review each and resolve as appropriate — these don't block the PR.

Editorial stances introduced by this PR

Superlative, ranking, or comparative language the diff adds. No verdict — a page's own framing isn't fact-checkable — but confirm each is a stance the docs should take, and that no agent-written rewrite introduced it unasked.

None — the extractor found no positioning or comparison language in this PR's added lines.

💡 Pre-existing issues in touched files (optional)

No pre-existing issues in touched files.

✅ Resolved since last review

No items resolved since the last review.

📜 Review history

  • 2026-09-04T03:43:50Z — Redirects-only change; all 168 added entries are well-formed, non-duplicate, and point at /dev/tutorials/ — no findings. (cff3c08)

  • Refresh this review — comment @claude #update-review. Say what you fixed, or which finding you dispute and why; both work in the same mention.
  • Ask for anything else — comment @claude with no hashtag (questions, one-off fixes). Leaves this review untouched.

Important

Please don't hide, resolve, or delete this comment! It breaks things!

📖 How pre-merge review works — the full lifecycle, short-circuits, and escape hatches.

@github-actions github-actions Bot added review:no-blockers Claude review completed cleanly; outstanding is empty and removed review:in-progress Claude review is currently running labels Sep 4, 2026
@pulumi-bot

Copy link
Copy Markdown
Collaborator

@cnunciato
cnunciato enabled auto-merge (squash) September 4, 2026 03:49
@cnunciato
cnunciato requested a review from a team September 4, 2026 15:26
@cnunciato
cnunciato merged commit ff29284 into master Sep 4, 2026
14 checks passed
@cnunciato
cnunciato deleted the cnunciato/guides-redirects-dev-tutorials branch September 4, 2026 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain:infra PR touches workflows, scripts, infra, Makefile, or build config review:no-blockers Claude review completed cleanly; outstanding is empty

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants