Skip to content

Conversation

@SaraDPH
Copy link
Contributor

@SaraDPH SaraDPH commented Dec 1, 2025

Proposed changes

Unreleased product version (optional)

Related issues (optional)

@SaraDPH SaraDPH requested a review from scottmparker December 1, 2025 22:33
@SaraDPH SaraDPH self-assigned this Dec 1, 2025
@SaraDPH SaraDPH added the area/blog Content issues on blog posts. label Dec 1, 2025
@pulumi pulumi deleted a comment from claude bot Dec 1, 2025
@pulumi-bot
Copy link
Collaborator

@SaraDPH
Copy link
Contributor Author

SaraDPH commented Dec 1, 2025

@claude take a look and tell me the issues

@claude
Copy link
Contributor

claude bot commented Dec 1, 2025

Claude finished @SaraDPH's task —— View job


Review Complete

I've reviewed the blog post and found several issues that need to be addressed:


Critical Issues

  1. Filename mismatch (content/blog/migrate-your-cdk-to-pulumi-with-neo/index.md:1)

    • The file is named index.md but should be named _index.md according to blogging conventions
    • This is the standard Hugo blog post naming pattern used throughout the repository
  2. Missing `` break (content/blog/migrate-your-cdk-to-pulumi-with-neo/index.md:73)

    • The `` comment exists but should appear after the first paragraph (around line 67-71)
    • Currently it appears after 4 paragraphs, which will make the blog home page preview too long
    • Best practice: Place it after the opening paragraph that ends with "Multi-cloud capabilities when you need them."

Style Guide Issues

  1. Heading capitalization (content/blog/migrate-your-cdk-to-pulumi-with-neo/index.md:75, 91, 111, 121, 155, 175, 194)

    • H2 headings should use sentence case, not title case
    • Incorrect: "Who Should Migrate", "What Changes", "Conversion vs. Migration", etc.
    • Correct: "Who should migrate", "What changes", "Conversion vs. migration", etc.
  2. Image path inconsistency (content/blog/migrate-your-cdk-to-pulumi-with-neo/index.md:143, 149)

    • Line 143: Uses absolute path /migrate-your-cdk-to-pulumi-with-neo/neo-plan-screenshot.png
    • Line 149: Uses relative path neo-preview-update-screenshot.png
    • Both should use relative paths: neo-plan-screenshot.png and neo-preview-update-screenshot.png
  3. Missing alt text on images (content/blog/migrate-your-cdk-to-pulumi-with-neo/index.md:143, 149)

    • Both images use generic "Placeholder Image" alt text
    • Should be descriptive, e.g.:
      • Line 143: ![](neo-plan-screenshot.png)
      • Line 149: ![](neo-preview-update-screenshot.png)

Content Issues

  1. Avoid words like "simple" (content/blog/migrate-your-cdk-to-pulumi-with-neo/index.md:86, 119, 161, 186)

    • Style guide discourages "simple" and "easy" as they judge difficulty
    • Line 86: "Your team has a deep investment..." (remove "simple" qualifier)
    • Line 119: "Start with conversion only" (remove "simple" from "simple multi-stack setups" on line 161)
    • Line 186: "State migration for straightforward, single-stack CDK applications..." (replace "simple" with "straightforward" or "basic")
  2. Inconsistent product name (content/blog/migrate-your-cdk-to-pulumi-with-neo/index.md:84)

    • Uses "CDK Pipelines" which is correct
    • But ensure consistency throughout (appears correct)
  3. Comments in frontmatter (content/blog/migrate-your-cdk-to-pulumi-with-neo/index.md:62)

    • Line 62 says "please remove these comments before submitting for review"
    • The comment block at lines 4-19, 22-25, 42-43, and 48-62 should be removed
  4. Empty social media fields (content/blog/migrate-your-cdk-to-pulumi-with-neo/index.md:58-59)

    • Twitter and LinkedIn fields are empty
    • Either populate them or remove the empty fields

Minor Issues

  1. Trailing spaces (Need to check with sed)

    • Run: sed -i '' 's/[[:space:]]*$//' content/blog/migrate-your-cdk-to-pulumi-with-neo/index.md to fix any trailing whitespace
  2. Code block formatting (content/blog/migrate-your-cdk-to-pulumi-with-neo/index.md:127-139)

    • The prompt/command block doesn't have a language specifier
    • Consider adding bash or text to the opening fence

Verification Needed

  1. Author ID (content/blog/migrate-your-cdk-to-pulumi-with-neo/index.md:31)

    • Uses neo-team as author
    • Verify this author file exists at /data/team/team/neo-team.toml
  2. Meta image (content/blog/migrate-your-cdk-to-pulumi-with-neo/index.md:25)

    • Verify meta.png exists and uses current logos
    • Verify it meets specs: 1200×628 PNG with opaque background

Summary

Must fix before merge:

  • Rename file to _index.md
  • Move `` break to after first paragraph
  • Fix all H2 headings to sentence case
  • Fix image paths to use relative paths consistently
  • Add descriptive alt text to images
  • Remove frontmatter comments
  • Handle empty social fields

Should fix:

  • Replace instances of "simple" with neutral terms
  • Check for trailing spaces
  • Add language specifier to code block

Verify:

  • Author file exists
  • Meta image meets specifications

@pulumi-bot
Copy link
Collaborator

@pulumi-bot
Copy link
Collaborator

@pulumi-bot
Copy link
Collaborator

meta image
meta description
temp date
@pulumi-bot
Copy link
Collaborator

@pulumi-bot
Copy link
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/blog Content issues on blog posts.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants