-
Notifications
You must be signed in to change notification settings - Fork 262
Add registry usage insights documentation and blog post #17531
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
base: master
Are you sure you want to change the base?
Conversation
Documentation ReviewI've reviewed PR #17531 which adds registry usage insights documentation and blog post. Overall, the content is well-structured and informative. Here are my findings: Critical Issues1. Broken documentation link in blog post (line 72) The blog post references 2. Duplicate image reference in blog post (line 41) The "Filter dropdown" section uses the same image ( Please verify if this is the correct image or if a different screenshot showing filters is needed. 3. Placeholder meta.png (SVG file) The Style and Content Issues4. Blog post: Inconsistent list formatting (lines 33-35) According to STYLE-GUIDE.md, all ordered list items should begin with 5. Documentation: Heading could be more specific (line 39) The heading "Usage tracking" in the docs is quite generic. Consider making it more specific to improve scanability: 6. Documentation: Improved section organization The "Component usage tracking" section was removed and integrated into "Usage tracking", but the flow could be clearer. Consider explicitly mentioning that usage tracking applies to both the Registry and Private Components tabs, since this was reorganized. Minor Issues7. Blog post: Consider rewording security scenario (line 65) The phrase "when a CVE drops" is somewhat informal for documentation. Consider: 8. Documentation: Menu reference needs verification (line 25) The image shows "Platform menu showing Registry tab", but please verify this matches the actual UI. The screenshot shows it's named Positive Aspects
Before Merging
Great work on documenting this feature! Once the critical issues are addressed, this will be ready to publish. Mention @claude if you'd like me to review again after making changes. |
|
Your site preview for commit 0f55cfd is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-17531-0f55cfd1.s3-website.us-west-2.amazonaws.com. |
|
Your site preview for commit add4824 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-17531-add48246.s3-website.us-west-2.amazonaws.com. |
|
@claude can you re-run your analysis? |
|
Claude encountered an error —— View job Re-running Documentation Review
|
Co-authored-by: Mark <mark@pulumi.com>
|
Your site preview for commit 5149d5b is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-17531-5149d5b3.s3-website.us-west-2.amazonaws.com. |
|
Your site preview for commit b45e7d9 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-17531-b45e7d9e.s3-website.us-west-2.amazonaws.com. |
|
Your site preview for commit 6bdb416 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-17531-6bdb4168.s3-website.us-west-2.amazonaws.com. |

Summary
Test plan