Drop self-contradicting "in minutes" claim from the Neo product page - #21569
workprentice[bot] wants to merge 1 commit into
Conversation
The "Faster provisioning" card claimed "in minutes" while its own proof point in the same sentence cited the Werner Enterprises result of 3 days to 4 hours. Drop the unverifiable time claim and let the substantiated figure carry the sentence. Fixes #21567
Reviewer's guide v1 — not for the authorTip This is the reviewer's guide. Work through the 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 wrongness that would matter here is a marketing speed claim the product can't back — and the card's own next sentence (3 days to 4 hours) contradicted "in minutes," so removing it resolves that. The factual-claim check and the frontmatter sweep both ran; no links, code samples, or sibling pages are involved. Review confidence:
|
Author action guide v1 — nothing blocks mergeNote Nothing here blocks merge — no open items need an answer from you. A human reviewer still approves the merge. This PR removes the "in minutes" phrase from one feature card on the Neo product page; the review checked the page's remaining factual claim, its frontmatter, and its links, and found nothing to fix. 🚨 Fix or disagreeNothing to fix — this section is empty. ❓ Questions for youNo open questions for you. 📎 Full evidence: verification trail, investigation log, review history. Review v1 · updated 2026-09-12T00:11:14Z · head commit bb72267 How to answerEvery 🚨 and ❓ item above needs one of these before merge:
The Please don't edit, hide, or delete this comment — it is the review's record. Full mechanics: CONTRIBUTING.md §AI-assisted contributions. |
|
Your site preview for commit bb72267 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-21569-bb722675.s3-website.us-west-2.amazonaws.com Changed pages: |
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
… 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>
Fixes #21567
The "Faster provisioning" card on the Neo product page claimed the deployment happens "in minutes," then immediately cited its own proof point of 3 days to 4 hours — a self-contradiction inside one sentence. "Complex multi-cloud deployments" is also an open-ended outcome whose duration depends entirely on what's being deployed, unlike a bounded first step (a quickstart, a single CLI action), so the "in minutes" framing was unverifiable on its own terms even setting the contradiction aside.
Before:
After:
The Werner Enterprises figure is unchanged and matches the same result cited on
/blog/ai-infrastructure-tools/and/what-is/what-is-agentic-infrastructure/, so this is a pure deletion — no new time claim introduced.Scope note: this same "in minutes" pattern turned up on three other pages during the sweep that also produced #21417/#21418 (the
/contact/fix). Those are tracked on their own cards and deliberately not bundled into this PR, so each can be reviewed independently; they can be consolidated into one PR later if reviewers prefer.🧠 This PR was created by workprentice.