Skip to content

Skip sparse trait#3363

Merged
jterapin merged 1 commit intoversion-3from
handle-sparse-trait
Mar 17, 2026
Merged

Skip sparse trait#3363
jterapin merged 1 commit intoversion-3from
handle-sparse-trait

Conversation

@jterapin
Copy link
Contributor

Some service models now include the sparse trait on list and map shapes. The v3 codegen in client_api_module doesn't support sparse semantics, but the presence of the trait in a model shouldn't break code generation.

This change makes codegen gracefully ignore the sparse trait so that affected services can still be generated.


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

  1. To make sure we include your contribution in the release notes, please make sure to add description entry for your changes in the "unreleased changes" section of the CHANGELOG.md file (at corresponding gem). For the description entry, please make sure it lives in one line and starts with Feature or Issue in the correct format.

  2. For generated code changes, please checkout below instructions first:
    https://github.com/aws/aws-sdk-ruby/blob/version-3/CONTRIBUTING.md

Thank you for your contribution!

Copy link
Contributor

@richardwang1124 richardwang1124 left a comment

Choose a reason for hiding this comment

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

LGTM!

@jterapin jterapin merged commit e202ca8 into version-3 Mar 17, 2026
28 checks passed
@jterapin jterapin deleted the handle-sparse-trait branch March 17, 2026 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants