Skip to content

fix: type unions in variadic position leads to invalid code-gen#3722

Merged
RomainMuller merged 1 commit intomainfrom
rmuller/type-union-vararg
Aug 24, 2022
Merged

fix: type unions in variadic position leads to invalid code-gen#3722
RomainMuller merged 1 commit intomainfrom
rmuller/type-union-vararg

Conversation

@RomainMuller
Copy link
Copy Markdown
Contributor

The code-gen for type unions was incorrect, as the value is typically
array-typed, and the generated code hence needs to treat the value as
such.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@RomainMuller RomainMuller self-assigned this Aug 24, 2022
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Aug 24, 2022
@RomainMuller RomainMuller force-pushed the rmuller/type-union-vararg branch 2 times, most recently from 70a806e to 5fd0289 Compare August 24, 2022 14:17
The code-gen for type unions was incorrect, as the value is typically
array-typed, and the generated code hence needs to treat the value as
such.
@RomainMuller RomainMuller force-pushed the rmuller/type-union-vararg branch from 5fd0289 to 6fede90 Compare August 24, 2022 15:06
@RomainMuller RomainMuller merged commit 93aec85 into main Aug 24, 2022
@RomainMuller RomainMuller deleted the rmuller/type-union-vararg branch August 24, 2022 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution/core This is a PR that came from AWS.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant