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.

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

Relax Converter validation for List and Map #688

Closed
wants to merge 1 commit into from

Conversation

pawelprazak
Copy link
Contributor

Fixes #629

Description

Fixes #629

Checklist

  • I have added tests that prove my fix is effective or that my feature works
  • Yes, there are changes in this PR that warrants bumping the Pulumi Service API version

@pawelprazak pawelprazak added the impact/no-changelog-required This issue doesn't require a CHANGELOG update label Jun 21, 2022
@t0yv0 t0yv0 self-requested a review June 28, 2022 14:32
@pawelprazak
Copy link
Contributor Author

After discussion with @t0yv0 over Zoom, we've decided to not go ahead with this workaround and go with a fix that preserves nested generic types

@pawelprazak pawelprazak deleted the pprazak/629/relax-converter-validation branch September 14, 2022 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact/no-changelog-required This issue doesn't require a CHANGELOG update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Serialization error when using helm to install a chart with Java
1 participant