-
Notifications
You must be signed in to change notification settings - Fork 19
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
Revamp config support #414
Conversation
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.
really nice!
the Java code needs to be de-go-ified ;)
sdk/jvm/pulumi/src/main/java/com/pulumi/core/internal/Codegen.java
Outdated
Show resolved
Hide resolved
sdk/jvm/pulumi/src/main/java/com/pulumi/core/internal/Codegen.java
Outdated
Show resolved
Hide resolved
sdk/jvm/pulumi/src/main/java/com/pulumi/core/internal/Codegen.java
Outdated
Show resolved
Hide resolved
sdk/jvm/pulumi/src/main/java/com/pulumi/core/internal/Codegen.java
Outdated
Show resolved
Hide resolved
sdk/jvm/pulumi/src/main/java/com/pulumi/core/internal/Codegen.java
Outdated
Show resolved
Hide resolved
return enumTypes | ||
} | ||
|
||
func (dg *defaultsGen) unEitherTypeShape(typeShape TypeShape) (bool, TypeShape, TypeShape) { |
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.
those un*TypeShape(...)
look like they should be on the TypeShape
itself, this one esp. looks useful for #138
...egen/testing/test/testdata/mini-awsnative/jvm/src/main/java/com/pulumi/awsnative/Config.java
Show resolved
Hide resolved
sdk/jvm/pulumi/src/main/java/com/pulumi/core/internal/Codegen.java
Outdated
Show resolved
Hide resolved
sdk/jvm/pulumi/src/main/java/com/pulumi/core/internal/Codegen.java
Outdated
Show resolved
Hide resolved
I'm doing a PR to this PR to help out with the Java part, and better understand limitations of our Environment and Config implementations we have right now. |
instead of making you do the changes from my review please take this one :) |
Code review for #414
Filed #420 bug discovered here. Also noting that Union and Either handling is a little sketchy, but at least seems to produce compiling code for now. We may need to revisit. There's a few further simplifications this opens up, including removing |
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
* Rebasing * Merging testdata * Add mini-azuread test case * Fix merge * Fix handling secret outputs * Accept changes * Fix generation of custom-named props like public_ * Implement secret() helper * [codegen] built and added added providers/pulumi-aws-native/ * [codegen] built and added added providers/pulumi-aws/ * [codegen] built and added added providers/pulumi-azure-native/ * [codegen] built and added added providers/pulumi-docker/ * [codegen] built and added added providers/pulumi-eks/ * [codegen] built and added added providers/pulumi-gcp/ * [codegen] built and added added providers/pulumi-google-native/ * [codegen] built and added added providers/pulumi-kubernetes/ * Pass lint * Code style improvements and cleanup Co-authored-by: Paweł Prażak <pprazak@virtuslab.com>
* Rebasing * Merging testdata * Add mini-azuread test case * Fix merge * Fix handling secret outputs * Accept changes * Fix generation of custom-named props like public_ * Implement secret() helper * [codegen] built and added added providers/pulumi-aws-native/ * [codegen] built and added added providers/pulumi-aws/ * [codegen] built and added added providers/pulumi-azure-native/ * [codegen] built and added added providers/pulumi-docker/ * [codegen] built and added added providers/pulumi-eks/ * [codegen] built and added added providers/pulumi-gcp/ * [codegen] built and added added providers/pulumi-google-native/ * [codegen] built and added added providers/pulumi-kubernetes/ * Pass lint * Code style improvements and cleanup Co-authored-by: Paweł Prażak <pprazak@virtuslab.com>
Description
Fixes # (issue)
Checklist