Skip to content

Revert "Add cloud-agnostic getting-started overview and connect-cloud page (#20428)" - #20638

Merged
cnunciato merged 1 commit into
masterfrom
cnunciato/revert-gs-content
Aug 3, 2026
Merged

Revert "Add cloud-agnostic getting-started overview and connect-cloud page (#20428)"#20638
cnunciato merged 1 commit into
masterfrom
cnunciato/revert-gs-content

Conversation

@cnunciato

@cnunciato cnunciato commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Reverts commit 14d9ba2 pending further discussion. (See internal thread for additional context.)

@cnunciato
cnunciato requested review from CamSoper, Copilot and tatcoo-pulumi and removed request for Copilot August 1, 2026 14:44
@cnunciato
cnunciato marked this pull request as ready for review August 1, 2026 14:45
@github-actions github-actions Bot added review:triaging Claude Triage is currently classifying the PR domain:docs PR touches technical docs review:in-progress Claude review is currently running and removed review:triaging Claude Triage is currently classifying the PR labels Aug 1, 2026
@github-actions

github-actions Bot commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Pre-merge Review — Last updated 2026-08-01T14:47:19Z

Tip

Summary: This PR reverts #20428, restoring content/docs/get-started/_index.md to its prior form (original meta_desc, the "Choose a cloud provider to get started:" lead-in, and none of the "Why Pulumi" / "How getting started works" / Terraform / ESC-and-Deployments sections) and deleting the Connect Your Cloud Account page at /docs/get-started/connect-cloud/. The failure mode that would matter to a reader is a dangling reference — prose, nav entries, or links elsewhere still pointing at the removed page or removed headings. No file anywhere in the repo references /docs/get-started/connect-cloud/, the removed page carried no aliases of its own, and its nav entry lived in its own frontmatter, so the revert is self-consistent. The one item left is the deleted URL itself, which was live on master and now 404s with no redirect. Passes run: full diff read, frontmatter alias/URL-collision sweep, a repo-wide inbound-link check for the deleted path, and claim verification over the retained prose (4 claims, 2 verified, none contradicted). Vale reported no style findings.

Review confidence:

Dimension Level Notes
mechanics HIGH
facts HIGH
links MEDIUM No repo-internal references to the deleted page remain; inbound external traffic to the removed URL can't be checked from CI.
Investigation log
  • Cross-sibling reads: 0 of 0 siblings (not in a templated section; no directory peers to compare against)
  • External claim verification: 2 of 4 claims verified (0 unverifiable, 0 contradicted) · 4 specialists (numerical, cross-reference, capability, framing); 0 cross-specialist corroborations · routed: 0 inline, 4 Pass 1, 0 Pass 2, 0 Pass 3.
  • Cited-claim spot-checks: not run (no cited claims)
  • Frontmatter sweep: ran on body + meta_desc
  • Temporal-trigger sweep: ran (recency words present in diff; spot-check in-review)
  • 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 1 0 0

🔍 Verification trail

5 claims extracted · 2 verified · 1 unverifiable · 0 contradicted
  • L3 in content/docs/get-started/_index.md "Pulumi provides step-by-step guides for creating, deploying, and managing infrastructure with Pulumi on the cloud using your favorite language." → ➖ not-a-claim (evidence: The claim text is a near-verbatim restatement of the page's own meta_desc frontmatter ("Step-by-step guides for creating, deploying, and managing infrastructure with Pulumi on the cloud using your favorite language.") describing what the…; source: repo:content/docs/get-started/_index.md)
  • L23 in content/docs/get-started/_index.md "Pulumi IaC optionally pairs with Pulumi Cloud to make managing infrastructure secure, reliable, and hassle-free." → ✅ verified (evidence: The claim is a direct quote from the page itself (L23: "Pulumi IaC is free, open source, and optionally pairs with Pulumi Cloud to make managing infrastructure secure, reliable, and hassle-free"), and is corroborated by the linked pulumi-c…; source: content/docs/iac/guides/basics/pulumi-cloud-vs-oss.md)
  • L25 in content/docs/get-started/_index.md "Users can choose a cloud provider to get started with Pulumi." → ✅ verified (evidence: The page text reads "Choose a cloud provider to get started:" followed by cards linking to AWS, Azure, Google Cloud, and Kubernetes get-started guides, directly matching the claim.; source: repo:content/docs/get-started/_index.md)
  • L25 in content/docs/get-started/_index.md "Getting started with Pulumi involves choosing a cloud provider." → ➖ not-a-claim (evidence: The page itself literally says "Choose a cloud provider to get started:" followed by cloud provider cards (AWS, Azure, GCP, Kubernetes) — this is a faithful description of the page's own content/design, not a falsifiable third-party techni…; source: repo:content/docs/get-started/_index.md)
  • L1 in content/docs/get-started/connect-cloud.md "the deleted page's URL /docs/get-started/connect-cloud/ is retired with no alias or redirect" → 🤷 unverifiable (evidence: The page is deleted outright and declares no aliases; a repo-wide search finds zero remaining references to the path, but whether the URL accumulated external inbound links during its time on master can't be determined from CI.; source: repo:content/docs/get-started/connect-cloud.md)

🚨 Outstanding in this PR

No outstanding findings in this PR.

⚠️ Low-confidence

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

  • [L1] content/docs/get-started/connect-cloud.md — the page is deleted, retiring the published URL /docs/get-started/connect-cloud/ with nothing to catch it. AGENTS.md treats a removed URL as needing a redirect ("Missing aliases on moved files break search rankings and external links"); since the file is gone, an alias can't live in its frontmatter, so the mechanism is an S3 redirect under /scripts/redirects/, pointing at the surviving overview page:

    /docs/get-started/connect-cloud/|https://www.pulumi.com/docs/get-started/
    

    Weighted low, not outstanding, because the page's stay on master was brief and no repo content links to it — if it never had time to be crawled or shared, the 404 costs nothing. Your call on whether it's worth the entry.

💡 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-08-01T14:47:19Z — Revert is self-consistent — no repo references to the removed page or headings remain; flagged the retired /docs/get-started/connect-cloud/ URL as lacking a redirect (7f87db1)

  • 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 Aug 1, 2026
@cnunciato
cnunciato enabled auto-merge (squash) August 1, 2026 14:52
@pulumi-bot

Copy link
Copy Markdown
Collaborator

Your site preview for commit 7f87db1 is ready! 🎉

http://www-testing-pulumi-docs-origin-pr-20638-7f87db12.s3-website.us-west-2.amazonaws.com

Changed pages:

@cnunciato
cnunciato merged commit 855e93e into master Aug 3, 2026
11 checks passed
@cnunciato
cnunciato deleted the cnunciato/revert-gs-content branch August 3, 2026 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain:docs PR touches technical docs 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