Skip to content

what-is-yaml: trim filler phrase#26

Closed
CamSoper wants to merge 2 commits intomasterfrom
test-pipeline/trivial-typo
Closed

what-is-yaml: trim filler phrase#26
CamSoper wants to merge 2 commits intomasterfrom
test-pipeline/trivial-typo

Conversation

@CamSoper
Copy link
Copy Markdown
Owner

One-word prose trim, single file, no links, no frontmatter.

Pipeline test: should be labeled review:trivial by triage and short-circuit the review job.

@CamSoper CamSoper marked this pull request as ready for review April 23, 2026 19:24
@CamSoper CamSoper force-pushed the test-pipeline/trivial-typo branch from 798cd13 to 7be492a Compare April 23, 2026 19:27
@CamSoper CamSoper marked this pull request as draft April 23, 2026 19:27
@CamSoper CamSoper marked this pull request as ready for review April 23, 2026 19:27
@CamSoper CamSoper marked this pull request as draft April 23, 2026 19:45
@CamSoper CamSoper marked this pull request as ready for review April 23, 2026 19:45
@claude claude Bot added review:docs PR touches technical docs review:trivial Tiny prose-only change; skips Claude review labels Apr 23, 2026
@claude
Copy link
Copy Markdown

claude Bot commented Apr 23, 2026

Claude Review — Last updated 2026-04-23T19:47:09Z

Status: 0 🚨 / 0 ⚠️ / 0 💡 / 0 ✅

No findings. The change removes the filler phrase "it's important to note that" from a single sentence in content/what-is/what-is-yaml.md; the resulting prose is tighter and reads cleanly. Nothing else in the diff to flag under the docs domain.

🚨 Outstanding in this PR

None.

⚠️ Low-confidence

None.

✅ Resolved since last review

None (initial review).

📜 Review history

  • 2026-04-23T19:47:09Z — initial review; no findings on a one-line prose trim (7be492a)

Mention @claude on this PR to request a re-review after addressing feedback or pushing new commits.

@claude claude Bot added the review:claude-ran Claude review has completed for this PR's current state label Apr 23, 2026
@CamSoper CamSoper force-pushed the master branch 6 times, most recently from 84d8cdc to 00a0780 Compare April 23, 2026 22:03
Fork-only tweak so claude.yml works without org-side ESC setup.
@claude retains all its capabilities (re-entrant reviews, Q&A,
make-changes on PRs) -- only difference is commits pushed with
GITHUB_TOKEN don't trigger downstream workflows, which is fine for
fork testing.

This commit is NOT for upstream. Origin/master and pulumi#18680
keep the ESC design. Do not cherry-pick.
@CamSoper CamSoper force-pushed the test-pipeline/trivial-typo branch from 7be492a to 967b26a Compare April 23, 2026 22:32
@github-actions github-actions Bot added the review:claude-stale New commits since last Claude review; refresh on next ready-transition or @claude mention label Apr 23, 2026
@CamSoper
Copy link
Copy Markdown
Owner Author

Test PR closed. The Claude PR review pipeline work is now tracked on pulumi#18680.

@CamSoper CamSoper closed this Apr 23, 2026
@CamSoper CamSoper deleted the test-pipeline/trivial-typo branch April 23, 2026 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review:claude-ran Claude review has completed for this PR's current state review:claude-stale New commits since last Claude review; refresh on next ready-transition or @claude mention review:docs PR touches technical docs review:trivial Tiny prose-only change; skips Claude review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant