Skip to content

Revise self-hosted Pulumi documentation content#17941

Merged
smithrobs merged 4 commits intomasterfrom
isaac-pulumi-patch-2
Mar 14, 2026
Merged

Revise self-hosted Pulumi documentation content#17941
smithrobs merged 4 commits intomasterfrom
isaac-pulumi-patch-2

Conversation

@isaac-pulumi
Copy link
Copy Markdown
Contributor

@isaac-pulumi isaac-pulumi commented Mar 12, 2026

Updated descriptions and titles for self-hosted Pulumi documentation to enhance clarity and accuracy.

Proposed changes

Position a proof of concept vs. download.
Address inaccurate pricing.

Unreleased product version (optional)

Related issues (optional)

Updated descriptions and titles for self-hosted Pulumi documentation to enhance clarity and accuracy.
@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Mar 12, 2026

Docs Review — PR #17941

Scope: content/product/self-hosted.md (11 additions, 11 deletions)

Overall this is a clean, focused PR that repositions the self-hosted page from a "free trial / download" framing to a "proof of concept / evaluation" framing. The changes are consistent and the intent is clear. A few items to address:

Issues

  1. meta_desc missing trailing period (line 4) — The original meta description ended with a period; the new one does not. For consistency and per Google style conventions (complete sentences), consider adding one:

    meta_desc: Learn how to operate Pulumi Cloud in your own cloud account or data center.
    
  2. "datacenter" → "data center" (line 4) — Per the Google Developer Documentation Style Guide, the standard spelling is "data center" (two words).

  3. Capability titles changed to Title Case (lines 18, 23, 28, 33) — The originals used sentence case (e.g., "Cloud engineering platform"), and the PR changes them to Title Case (e.g., "Cloud Engineering Platform"). The same applies to the questions.title ("Talk to a Human"). These are YAML card/section titles rather than markdown H2+ headings, so the style guide's "sentence case for H2+" rule doesn't strictly apply. Title Case is reasonable for UI card labels and is consistent with how the overview.title and trial.title fields were already styled. Just flagging for awareness — no change needed unless the team has a preference.

Looks Good

  • The anchor link [Request a Proof of Concept](#self-hosted-trial) still resolves correctly since the trial: YAML key is unchanged.
  • Pricing description update accurately reflects the new positioning (additional license + guided POC vs. included + free 30-day trial).
  • No file moves, no broken links, no infrastructure changes.
  • Frontmatter fields are present and well-formed.

Mention me (@claude) if you'd like additional reviews or automated fixes.

@pulumi-bot
Copy link
Copy Markdown
Collaborator

@isaac-pulumi isaac-pulumi requested a review from smithrobs March 12, 2026 19:08
@smithrobs
Copy link
Copy Markdown
Contributor

smithrobs commented Mar 12, 2026

@claude would you address issues 1 and 2 in your review?

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Mar 12, 2026

Claude finished @smithrobs's task in 33s —— View job


Docs Review — PR #17941

Addressed issues #1 and #2 in commit 2ef0ea9.

Tasks

Changes made

In content/product/self-hosted.md line 5:

- meta_desc: Learn how to operate Pulumi Cloud in your own cloud account or datacenter
+ meta_desc: Learn how to operate Pulumi Cloud in your own cloud account or data center.

Both fixes were applied in a single line update — trailing period added and "datacenter" corrected to "data center".
| Branch

Co-authored-by: Rob Smith <smithrobs@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@smithrobs smithrobs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for the overall glow up of the page!

@pulumi-bot
Copy link
Copy Markdown
Collaborator

@smithrobs smithrobs merged commit 5b0e620 into master Mar 14, 2026
9 checks passed
@smithrobs smithrobs deleted the isaac-pulumi-patch-2 branch March 14, 2026 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants