-
Notifications
You must be signed in to change notification settings - Fork 258
Additional languages for the Directly Executed Modules blog: C#, YAML, Java #15397
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
|
Your site preview for commit e9273e6 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-15397-e9273e67.s3-website.us-west-2.amazonaws.com. |
|
Your site preview for commit 45b1502 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-15397-45b1502f.s3-website.us-west-2.amazonaws.com. |
|
Your site preview for commit 0b7b7a9 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-15397-0b7b7a97.s3-website.us-west-2.amazonaws.com. |
| {{% /choosable %}} | ||
|
|
||
| {{% choosable language python %}} | ||
| ```bash |
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.
Addressing comments from previous PR here.
|
|
||
| {{% choosable language python %}} | ||
| ```python | ||
| import * as pulumi from "@pulumi/pulumi"; |
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.
Fixing erroneously copied TypeScript code from previous PR here.
|
Your site preview for commit 1ea06d6 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-15397-1ea06d60.s3-website.us-west-2.amazonaws.com. |
justinvp
left a comment
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.
Aside from the Java rendering issue, LGTM
|
Your site preview for commit b20c040 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-15397-b20c0402.s3-website.us-west-2.amazonaws.com. |
|
That fixed it! Thank you. |

Proposed changes
Following up on #15395 the PR adds example translations to cover C#, YAML and Java experience. None of these languages currently support terraformConfig() method and the examples for this method are omitted for these languages. The feature is tracked in pulumi/pulumi-terraform-bridge#3131
Unreleased product version (optional)
Related issues (optional)