Skip to content
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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

AWS Fargate example generates non-compiling Go code #8440

Closed
t0yv0 opened this issue Nov 17, 2021 · 0 comments 路 Fixed by #11480
Closed

AWS Fargate example generates non-compiling Go code #8440

t0yv0 opened this issue Nov 17, 2021 · 0 comments 路 Fixed by #11480
Assignees
@iwahbe
Labels
area/codegen SDK-gen, program-gen, convert kind/bug Some behavior is incorrect or out of spec language/go resolution/fixed This issue was fixed
Milestone
0.81

Comments

@t0yv0
Copy link
Member

t0yv0 commented Nov 17, 2021

Hello!

  • Vote on this issue by adding a 馃憤 reaction
  • To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already)

Issue details

The test has to skip Go compilation in program_driver.go because the generated code is not valid Go. Adding an issue to track.

Steps to reproduce

Expected:
Actual:

@t0yv0 t0yv0 added the kind/bug Some behavior is incorrect or out of spec label Nov 17, 2021
@mikhailshilkov mikhailshilkov added area/codegen SDK-gen, program-gen, convert and removed area/examples labels Jul 22, 2022
@iwahbe iwahbe mentioned this issue Nov 29, 2022
Merged
@iwahbe iwahbe self-assigned this Nov 29, 2022
@iwahbe iwahbe added this to the 0.81 milestone Nov 29, 2022
@bors bors bot closed this as completed in 9086bf9 Nov 29, 2022
@pulumi-bot pulumi-bot added the resolution/fixed This issue was fixed label Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@iwahbe iwahbe

Labels
area/codegen SDK-gen, program-gen, convert kind/bug Some behavior is incorrect or out of spec language/go resolution/fixed This issue was fixed
Projects
None yet
Milestone
0.81
Development

Successfully merging a pull request may close this issue.

Convert invoke results to ouputs when needed pulumi/pulumi
5 participants
@t0yv0 @mikhailshilkov @emiliza @iwahbe @pulumi-bot