Skip to content

Conversation

@SaraDPH
Copy link
Contributor

@SaraDPH SaraDPH commented Oct 27, 2025

Proposed changes

Unreleased product version (optional)

Related issues (optional)

@claude
Copy link
Contributor

claude bot commented Oct 27, 2025

Review of PR #16399

Thank you for improving the event description with specific time details. I found a few minor issues to address:

Issues Found

Line 20: Time format and trailing spaces

  1. UTC format: The format "18:00 (UTC +0)" is non-standard. Use "18:00 UTC" instead.
  2. Trailing spaces: The line has trailing whitespace that should be removed per repository standards.

Suggested Fix

  description: Join us on November 5 at 10:00 AM PT / 18:00 UTC for Pulumi Policies: Get Clean and Stay Clean Automatically. Watch it in action. Live demo + Q&A.

This change:

  • Uses standard "UTC" instead of "(UTC +0)"
  • Removes trailing spaces
  • Maintains clarity about the event time

To fix trailing spaces across files in the future, you can use:

sed -i '' 's/[[:space:]]*$//' content/product/pulumi-insights.md

Mention @claude if you'd like me to review again after making changes or if you need any clarification.

@adamgordonbell
Copy link
Contributor

@claude fixed? What now

@claude
Copy link
Contributor

claude bot commented Oct 27, 2025

Claude Code is working…

I'll analyze this and get back to you.

View job run

@SaraDPH
Copy link
Contributor Author

SaraDPH commented Oct 27, 2025

@adamgordonbell it still gives an error... it says a suggestion sed -i '' 's/[[:space:]]*$//' content/product/pulumi-insights.md but I don't know where to place it.

@pulumi-bot
Copy link
Collaborator

@adamgordonbell adamgordonbell self-requested a review October 28, 2025 11:50
@SaraDPH SaraDPH enabled auto-merge (squash) October 28, 2025 13:19
@pulumi-bot
Copy link
Collaborator

@SaraDPH SaraDPH merged commit 1d29ad3 into master Oct 28, 2025
8 checks passed
@SaraDPH SaraDPH deleted the update-launch-copy branch October 28, 2025 17:00
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.

4 participants