Skip to content

Conversation

@SaraDPH
Copy link
Contributor

@SaraDPH SaraDPH commented Nov 5, 2025

Proposed changes

Unreleased product version (optional)

Related issues (optional)

@claude
Copy link
Contributor

claude bot commented Nov 5, 2025

Documentation Review

I've reviewed the changes to the banner announcement in layouts/partials/banner.html.

Summary

This PR updates the top navigation banner to reflect that the Neo policy automation feature has launched. The changes are appropriate and look good overall.

Issues Found

Minor punctuation issue in line 1:

The em dash should have a space before it to match standard formatting for em dashes used as parenthetical breaks in English text.

{{ $summary := "Neo just got smarter about infrastructure policy automation — " }}

Verified

Link validation: The target URL /product/insights-governance/#video is valid. The #video anchor exists in the layout template at layouts/product/insights-governance.html:23.

File formatting: File correctly ends with a newline.

Grammar and clarity: The updated text is clear and action-oriented.

Call-to-action: "Watch the Launch" is more compelling than "Save the Date" for a post-launch announcement.


Mention me (@claude) if you'd like me to review any additional changes or have questions!

@pulumi-bot
Copy link
Collaborator

@SaraDPH SaraDPH merged commit 4d123eb into master Nov 5, 2025
9 checks passed
@SaraDPH SaraDPH deleted the Update-banner branch November 5, 2025 17:11
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