-
Notifications
You must be signed in to change notification settings - Fork 225
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
Add guide for Pulumi CDK #13590
Add guide for Pulumi CDK #13590
Conversation
This PR adds a guide to using Pulumi CDK. This seemed like the best place to put this content, but if there is a better place let me know! I plan on creating a separate migration guide which I will then update this to link to. re #250
Your site preview for commit 067915f is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-13590-067915ff.s3-website.us-west-2.amazonaws.com. |
I think this could use a link to construct hub and some top constructs close to the top of the article! Otherwise LGTM. Needs @pulumi/docs to pipe in where this should go though. |
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.
Looks great! Left some minor comments
At this point the lookups have been performed and you should be able to run | ||
Pulumi commands without errors. | ||
|
||
## Using Pulumi Policy Packs |
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.
Nice!
Co-authored-by: Florian Stadler <[email protected]>
Your site preview for commit 618434d is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-13590-618434d2.s3-website.us-west-2.amazonaws.com. |
Your site preview for commit 01a8b9b is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-13590-01a8b9b6.s3-website.us-west-2.amazonaws.com. |
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.
LGTM!
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.
Looks good. Thanks @corymhall !
This PR adds a guide to using Pulumi CDK. This seemed like the best place to put this content, but if there is a better place let me know!
I plan on creating a separate migration guide which I will then update this to link to.
re #250
Proposed changes
Unreleased product version (optional)
Related issues (optional)