-
Notifications
You must be signed in to change notification settings - Fork 255
Remove Copilot documentation and redirect to Neo product page #16439
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Removes the standalone Copilot article from the AI documentation section and redirects all Copilot URLs to the Neo product page. Copilot has been superseded by Neo, so the documentation should point users to the current product. Changes: - Removed /docs/ai/copilot/_index.md and associated images - Added copilot URL aliases to /product/neo.md to redirect to Neo - Removed copilot aliases from /docs/ai/_index.md - This removes "Copilot" from the Infrastructure AI sidebar menu 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
|
Your site preview for commit 518497c is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16439-518497c3.s3-website.us-west-2.amazonaws.com. |
Review of PR #16439This PR correctly removes the standalone Copilot documentation and redirects to the Neo product page. The implementation follows best practices for file removal and URL preservation. ✅ Strengths
📝 ObservationsInternal documentation links: Several documentation pages still reference the old Copilot URLs (e.g.,
Status: This is acceptable. The aliases in neo.md will handle these redirects automatically. However, per AGENTS.md guidelines, you may want to update internal links in find content/docs -name "*.md" -exec sed -i 's|/docs/ai/copilot/|/product/neo|g' {} +Blog posts: Multiple blog posts reference old Copilot URLs. Per AGENTS.md, blog posts are historical documents and should NOT be updated. The aliases will handle redirects. ✅ ✅ Verification
SummaryThis PR is ready to merge. The aliases ensure all old Copilot URLs redirect properly to Neo, preserving SEO and preventing broken links. Internal documentation link updates are optional but not required since aliases handle redirects. Mention @claude if you'd like me to update the internal documentation links or review any other changes. |
Documentation ReviewThis PR successfully removes the standalone Copilot documentation and redirects all Copilot URLs to the Neo product page. The implementation follows the repository's file deletion and aliasing guidelines. ✅ What's Good
📝 Suggestion: Update Internal LinksAccording to The following files in content/docs/insights/discovery/_index.md:5 content/docs/deployments/get-started/_index.md:15 (approximate line) content/docs/deployments/get-started/what-is-it.md:23 (approximate line) content/docs/iac/comparisons/cloud-template-transpilers/aws-cdk/_index.md:32 (approximate line) NoteThe API reference file Overall, this is a well-executed redirect. The alias implementation is correct and will preserve SEO. The internal link updates are optional but recommended for cleaner documentation and better performance (avoiding unnecessary redirects). Mention me (@claude) if you'd like me to help make these internal link updates or if you have questions about the review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect, thank you @arturl!
Removes the standalone Copilot article from the AI documentation section and redirects all Copilot URLs to the Neo product page. Copilot has been superseded by Neo, so the documentation should point users to the current product.
Changes:
Fixes https://github.com/pulumi/pulumi.ai/issues/2155