-
Notifications
You must be signed in to change notification settings - Fork 256
Enhance documentation structure for Pulumi ESC getting started guides #16533
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 #16533 which enhances the Pulumi ESC getting started guide structure. Overall, this is a well-executed documentation improvement that makes it easier for users to navigate based on their familiarity with Pulumi. Below are my findings: Issues Found1. Missing article in heading (content/docs/esc/get-started/_index.md:32)Location: Issue: The heading "Choose your path" should include the article "the" for proper grammar. Suggestion: ## Choose your pathShould be: ## Choose the right pathOr alternatively: ## Choose your learning path2. Incomplete sentence structure (content/docs/esc/get-started/integrate-with-pulumi-iac.md:78)Location: Issue: The sentence "Share AWS OIDC credentials across multiple stacks:" uses a colon to introduce a code block, but the context could be clearer. Consider rewording for better flow. Current: Share AWS OIDC credentials across multiple stacks:Suggested: To share AWS OIDC credentials across multiple stacks, configure your ESC environment like this:3. Inconsistent section header capitalization (content/docs/esc/get-started/integrate-with-pulumi-iac.md:42)Location: Issue: "pulumiConfig" in the explanatory text should be formatted as code since it's a YAML key. Current: The `pulumiConfig` block maps ESC values to Pulumi configuration keys.This is actually correct - good job using code formatting! No change needed. Positive Observations
Content Accuracy
Style Compliance
RecommendationThe PR is in good shape. The two issues noted above are minor and don't block merging, but would improve the documentation quality if addressed. Mention @claude if you'd like me to review additional changes or help with fixes. |
|
Your site preview for commit 13344f3 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16533-13344f39.s3-website.us-west-2.amazonaws.com. |
|
Your site preview for commit 3950013 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16533-3950013c.s3-website.us-west-2.amazonaws.com. |
|
Your site preview for commit 6f7f4e3 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16533-6f7f4e3a.s3-website.us-west-2.amazonaws.com. |
…th Pulumi IaC, and setting up OIDC - Introduced new "Get Started" - Created a new index page for guides on Pulumi ESC. - Added detailed guides for integrating external secrets from AWS, Azure, and GCP. - Introduced a guide on importing environments to share configuration across teams. - Developed a guide on managing secrets, including storing and retrieving them. - Added instructions for running commands with `esc run` to inject secrets into scripts. - Created a guide for setting up OIDC to generate dynamic cloud credentials. - Updated references in existing documentation to point to new guides.
|
Your site preview for commit c4e9c68 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16533-c4e9c682.s3-website.us-west-2.amazonaws.com. |
…tes, and enhance instructions for OIDC setup and environment importing
|
Your site preview for commit 7b45727 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16533-7b457276.s3-website.us-west-2.amazonaws.com. |
- Created a new script `add_borders.py` that adds 1px #CCCCCC borders to PNG images referenced in markdown files. - Implemented functionality to check for existing borders and avoid duplication. - Added metadata to processed images to track border addition. - Included a `Pipfile` for dependency management, requiring `pillow` and `click`. - Added command-line interface for user interaction, supporting dry-run mode.
|
Your site preview for commit 8e964bf is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16533-8e964bf0.s3-website.us-west-2.amazonaws.com. |
This PR restructures the Pulumi ESC documentation to improve discoverability and user experience.
Key Changes
Getting Started:
New Guides Section:
/docs/esc/guides/with 6 self-contained guides:Concepts Enhancement:
Navigation & SEO:
git mvfor moved contentLink Fixes:
Fixes #15784