Skip to content

Conversation

@SaraDPH
Copy link
Contributor

@SaraDPH SaraDPH commented Oct 22, 2025

Placing the fireside discussion on-demand

@claude
Copy link
Contributor

claude bot commented Oct 22, 2025

Review Summary

This PR updates the event page to reflect the on-demand format. The changes are mostly good, but there are a few issues to address:

Issues Found

Line 50 (content/events/cloud-automation-intelligent-ops/index.md:50):
The AWS link includes a hash anchor #video that may not exist on the target page:

Join Pulumi and [AWS](https://www.pulumi.com/aws#video) for an exclusive chat

Suggestion:

Join Pulumi and [AWS](/aws/) for an exclusive chat

Line 73 (content/events/cloud-automation-intelligent-ops/index.md:73):
There's an extra leading space in the topics array before "AWS Bedrock":

topics:  ["AI", "Automation", " AWS Bedrock", "DevOps"]

Suggestion:

topics:  ["AI", "Automation", "AWS Bedrock", "DevOps"]

Minor Style Consideration

Lines 4, 50, and 89: The change from "fireside chat" to "chat" makes the description more concise, which is good. However, "chat" might feel slightly informal for enterprise content. Consider alternatives like "discussion" or "conversation" if you want a more professional tone while maintaining brevity.

What Looks Good

  • Duration accurately updated to 60 minutes for on-demand format
  • Gated status correctly changed to false
  • Meagan Cojocar's title properly updated
  • Pulumi Neo link is correct and resolves properly
  • DevOps tag appropriately added to topics

Mention @claude if you'd like additional reviews or fixes after addressing these issues.

@SaraDPH SaraDPH marked this pull request as ready for review October 22, 2025 20:32
@claude
Copy link
Contributor

claude bot commented Oct 22, 2025

Documentation Review

Thank you for updating this event page for on-demand viewing. I've reviewed the changes and found the following issues:

Issues Found

1. Inconsistent date references (Line 41, 85, 86, 87)

The event dates reference October 2025 (2025-10-09), which appears to be incorrect given that today is October 2025. This should likely be January 2025 (2025-01-09) based on typical webinar scheduling patterns. Please verify the correct date.

# Current (appears incorrect):
sortable_date: 2025-10-09T12:00:00-04:00
start_date: 2025-10-09T12:00:00-04:00
end_date: 2025-10-09T13:30:00-04:00

# If this should be January 2025:
sortable_date: 2025-01-09T12:00:00-04:00
start_date: 2025-01-09T12:00:00-04:00
end_date: 2025-01-09T13:30:00-04:00

2. Trailing space in topics tag (Line 73)

There's an extra space after "Automation" in the topics list that should be removed for consistency.

# Current:
topics:  ["AI", "Automation", "AWS Bedrock", "DevOps"]

# Should be:
topics: ["AI", "Automation", "AWS Bedrock", "DevOps"]

3. Missing newline at end of file (Line 90)

According to AGENTS.md, all markdown files must end with a newline. Please add a blank line after the final ---.

4. Style: Inconsistent hyphenation in description (Line 89)

The event_data.description uses "next-generation" (hyphenated) while main.description (line 51) also uses "next-generation" consistently. However, the phrase "AI-powered" is only used in main.description and not in event_data.description. For consistency, consider whether these descriptions should be more aligned.

Positive Changes

✓ Correctly updated gated: false to enable on-demand viewing
✓ Added the YouTube embed URL
✓ Updated duration from 90 to 60 minutes
✓ Changed "fireside chat" to "discussion" for more professional tone
✓ Added proper links to AWS and Pulumi Neo in the description
✓ Updated Meagan's title to reflect current role
✓ Removed extra space from "AWS Bedrock" tag and added "DevOps" tag
✓ Added new meta_image file


Please address the issues above, particularly verifying the event date. Mention me (@claude) if you'd like me to review again after making changes.

@SaraDPH SaraDPH self-assigned this Oct 22, 2025
@SaraDPH
Copy link
Contributor Author

SaraDPH commented Oct 22, 2025

I don't understand why it failed, can you please take a look @CamSoper @adamgordonbell ?

@CamSoper
Copy link
Contributor

@SaraDPH It's nothing you did. The build is broken at the moment, along with some of the live site.

@pulumi-bot
Copy link
Collaborator

@SaraDPH SaraDPH merged commit c8af5f9 into master Oct 22, 2025
15 of 17 checks passed
@SaraDPH SaraDPH deleted the Webinar-Cloud-Ops-AWS branch October 22, 2025 21:28
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