Skip to content

Conversation

0marperez
Copy link
Contributor

Issue #

closes #1129

Description of changes

Adds a test to make sure we don't regress.
See: aws/aws-sdk-kotlin#1526

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

@0marperez 0marperez added the no-changelog Indicates that a changelog entry isn't required for a pull request. Use sparingly. label Feb 13, 2025

This comment has been minimized.

1 similar comment

This comment has been minimized.

@0marperez 0marperez marked this pull request as ready for review February 13, 2025 20:24
@0marperez 0marperez requested a review from a team as a code owner February 13, 2025 20:24
Comment on lines +228 to +229
@Test
fun `it compiles models with unions with members that have the same name as the union`() {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

docs: add a comment with the original GitHub issue that inspired this test case

Copy link

Affected Artifacts

No artifacts changed size

@0marperez 0marperez merged commit ad4d2a3 into main Feb 14, 2025
16 checks passed
@0marperez 0marperez deleted the test-union-member-name-same-as-union branch February 14, 2025 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog Indicates that a changelog entry isn't required for a pull request. Use sparingly.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Codegen doesn't properly handle unions with a member that has the same name as the union
3 participants