Skip to content

Fix unbounded "idea to production in minutes" claim on IDP page - #21571

Open
workprentice[bot] wants to merge 1 commit into
masterfrom
fix/idp-unbounded-minutes-claim
Open

workprentice[bot] wants to merge 1 commit into
masterfrom
fix/idp-unbounded-minutes-claim

Conversation

@workprentice

@workprentice workprentice Bot commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

What

Fixes #21568

The "Ship faster" card on /product/internal-developer-platforms/ claimed engineers go from idea to production "in minutes." "Idea to production" has no fixed scope (it spans requirements, design, review, and rollout), so the claim is unverifiable — the same defect class fixed on /contact/ in #21417/#21418.

Change

- Golden paths and reusable components get engineers from idea to production in minutes.
+ Golden paths and reusable components let engineers ship approved infrastructure without waiting on the platform team.

One line, one file. Keeps a concrete capability statement consistent with the rest of the page (Components, Self-Service, Policies sections) without introducing a number that would need customer attribution — the three-card grid has no room for that. See the linked issue for the survey of existing case-study stats considered and why none fit here.

Verification

  • git diff --check: clean, no whitespace issues
  • YAML front matter re-parses correctly after the edit; confirmed the card's description field reads as intended
  • Confirmed via git grep this is the only occurrence of the defective phrasing left in the repo (other "idea to production" hits in blog/what-is content carry no time bound and are out of scope)

🧠 This PR was created by workprentice.

The "Ship faster" card on /product/internal-developer-platforms/
claimed engineers go from idea to production "in minutes," an
open-ended, unverifiable end-to-end time claim (idea to production
has no fixed scope). Same defect class as the /contact/ fix in
#21417/#21418.

Replaced with a concrete capability statement consistent with the
rest of the page, without introducing a number that would need
customer attribution the card grid has no room for.

Fixes #21568
@github-actions github-actions Bot added the review:triaging Claude Triage is currently classifying the PR label Sep 12, 2026
@github-actions github-actions Bot added domain:website PR touches marketing, pricing, legal, or competitive landing pages review:in-progress Claude review is currently running and removed review:triaging Claude Triage is currently classifying the PR labels Sep 12, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Reviewer's guide v1 — not for the author

Tip

This is the reviewer's guide. Work through the ⚠️ checklist below, then approve — approving asserts only that the ⚠️ items looked right to you. Machine-verified this run: links, shortcodes, page metadata, and every claim marked verified (receipts on the evidence page). Code samples are read, not compiled.

PR author: your to-do list is the other review comment, "Author action guide" — nothing on this card is yours.

Note

What this PR changes:

  • The "Ship faster" card on /product/internal-developer-platforms/ no longer promises engineers get "from idea to production in minutes."
  • That card now describes the capability instead: golden paths and reusable components let engineers ship approved infrastructure without waiting on the platform team.

The wrongness that would matter here is a promise the page can't keep — a timing figure with no defined start or end point that a prospect could hold us to. The page's frontmatter and metadata were swept and the one changed sentence was assessed for factual content; it makes no measurable assertion, so there was nothing to fact-check.

Review confidence:

Dimension Level Notes
mechanics HIGH
facts HIGH Nothing to verify — the replacement sentence describes the product's own design, not a measurable outcome.

⚠️ Check these before approving

No findings to check — but the editorial stances below still need a human eye.

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.

✅ What you can rubber-stamp

  • Facts: no factual claims found in the changed lines.
  • Mechanics: frontmatter sweep ran; no internal links added.
  • Style: 0 advisory suggestion(s) left with the author; never blocking.

💡 Pre-existing issues in touched files: 0 — details on the evidence page.

📎 Full evidence: verification trail, investigation log, review history.

Review v1 · updated 2026-09-12T00:11:37Z · head commit f79fd50


For the reviewer: the ⚠️ items above are the minutes that matter — the receipts for everything machine-verified are on the evidence page. The author's open items live on their own card (the comment headed "Author action guide"); while any are open, a Waiting on the author list above tracks them, and merge stays blocked until they're answered. If something here seems off, comment on the PR — @claude <your point> #update-review re-adjudicates with your input.

@github-actions

Copy link
Copy Markdown
Contributor

Author action guide v1 — nothing blocks merge

Note

Nothing here blocks merge — no open items need an answer from you. A human reviewer still approves the merge.

This PR rewrites one "Ship faster" card on the internal developer platforms product page to drop the unbounded "idea to production in minutes" timing claim; the review checked the page's metadata and the replacement wording and found nothing that needs your attention.

🚨 Fix or disagree

Nothing to fix — this section is empty.

❓ Questions for you

No open questions for you.

📎 Full evidence: verification trail, investigation log, review history.

Review v1 · updated 2026-09-12T00:11:37Z · head commit f79fd50


How to answer

Every 🚨 and ❓ item above needs one of these before merge:

  1. Fix it — push the change. If your push lines up with the flagged lines, this card shows a 🔄 banner within a minute and then refreshes itself. No banner? Comment:

    @claude I pushed a fix for F1 #update-review
    
  2. Disagree with it — say which item and why; the review re-checks with your input:

    @claude F2: <your reasoning — e.g. where the figure comes from> #update-review
    

    Either way your answer counts: the review marks the item resolved, or keeps it with a 🛡️ note for your human reviewer to weigh — it stops blocking merge in both cases.

  3. Accept it as-is — you own the PR; a one-line reason is a valid answer, and your reviewer sees it beside the finding:

    @claude F2: accepting as-is — <your reason> #update-review
    

    To accept every open item at once: @claude accepting all open items — <reason> #update-review.

The #update-review hashtag matters — it routes your reply to this review, and (besides pushing a fix) it is the only reply that unblocks merge. A bare @claude gets you ad-hoc help and leaves this card — and the merge block — untouched.

Please don't edit, hide, or delete this comment — it is the review's record. Full mechanics: CONTRIBUTING.md §AI-assisted contributions.

@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 12, 2026
@pulumi-bot

Copy link
Copy Markdown
Collaborator

@workprentice
workprentice Bot enabled auto-merge (squash) September 12, 2026 00:27
@CamSoper
CamSoper requested a review from cnunciato September 14, 2026 22:59
CamSoper added a commit that referenced this pull request Sep 17, 2026
checks_rollup counted every check run on the head commit. The API's
default filter=latest is per check suite, and each workflow run is its
own suite, so runs a concurrency group cancelled stayed in the list next
to the run that replaced them. `cancelled` counts as failed, so #21598
showed red CI on the board, got no action, and would have failed the
stamp preflight, with Sentinel green. #21569 and #21571 had the same
false red.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018NheEZ1S5NJbV6FCHkHPgc
CamSoper added a commit that referenced this pull request Sep 17, 2026
… runs don't turn CI red (#21682)

* pr-review: keep resolved findings out of a send-back

A judgment with no `ask` fell back to its `decision`, which is the
approver's own question. On a --request-changes that meant a finding the
approver had accepted or marked not-applicable went to the author as
"Move the role section to its own page?", inviting a change the approver
had decided against.

ask_lines now skips a resolved judgment unless it carries an explicit
`ask`, and no longer falls back to the card's open items once the judge
step has run. --request-changes refuses a row with nothing left to ask.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018NheEZ1S5NJbV6FCHkHPgc

* pr-review: count only the newest run of each check

checks_rollup counted every check run on the head commit. The API's
default filter=latest is per check suite, and each workflow run is its
own suite, so runs a concurrency group cancelled stayed in the list next
to the run that replaced them. `cancelled` counts as failed, so #21598
showed red CI on the board, got no action, and would have failed the
stamp preflight, with Sentinel green. #21569 and #21571 had the same
false red.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018NheEZ1S5NJbV6FCHkHPgc

* pr-review: dedupe check runs per workflow, not per name

The previous commit kept the newest run of each check name. Job names
repeat across workflows: pull-request.yml and testing-build-and-deploy.yml
both run "Install deps and build site", master's required check. A
failed PR build followed by a passing --deploy run read as green, so the
stamp preflight could pass and an admin merge go through.

The rollup now keys on (workflow, name), mapping each run's check suite
to its workflow through the Actions runs for the head SHA. A suite it
can't map keys on the suite itself, which errs red.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018NheEZ1S5NJbV6FCHkHPgc

---------

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain:website PR touches marketing, pricing, legal, or competitive landing pages impact/no-changelog-required review:no-blockers Claude review completed cleanly; outstanding is empty

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unbounded "idea to production in minutes" claim on /product/internal-developer-platforms/

2 participants