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.

Sign up for GitHub

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

Jump to bottom

Nested server structure member shapes targeting simple shapes with default trait #2352

Merged
merged 10 commits into from
Feb 17, 2023
Merged

Nested server structure member shapes targeting simple shapes with default trait #2352

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
e4d9a09
Nested server structure member shapes targeting simple shapes with `@…
82marbag Feb 13, 2023
d4432a7
Add changelog entry
82marbag Feb 13, 2023
d2827fa
Update CHANGELOG
82marbag Feb 16, 2023
2f2ad58
Merge branch 'main' into 2343
82marbag Feb 16, 2023
168e5ed
Merge branch 'main' into 2343
82marbag Feb 17, 2023
8d1689b
Add unit test
82marbag Feb 17, 2023
8bde7ec
Add integration test
82marbag Feb 17, 2023
62702ed
Change method
82marbag Feb 17, 2023
ec0b287
Include comment to describe the test
82marbag Feb 17, 2023
cd06810
Merge branch 'main' into 2343
82marbag Feb 17, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
6 changes: 6 additions & 0 deletions CHANGELOG.next.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,3 +10,9 @@
# references = ["smithy-rs#920"]
# meta = { "breaking" = false, "tada" = false, "bug" = false, "target" = "client | server | all"}
# author = "rcoh"

[[smithy-rs]]
message = "Mark unconstrained structures with optional _or_ default members."
82marbag marked this conversation as resolved.
Show resolved Hide resolved
references = ["smithy-rs#2352"]
82marbag marked this conversation as resolved.
Show resolved Hide resolved
meta = { "breaking" = false, "tada" = false, "bug" = true, "target" = "server"}
author = "82marbag"
2 changes: 1 addition & 1 deletion ...n-server/src/main/kotlin/software/amazon/smithy/rust/codegen/server/smithy/Constraints.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ fun Shape.isDirectlyConstrained(symbolProvider: SymbolProvider): Boolean = when
// The only reason why the functions in this file have
// to take in a `SymbolProvider` is because non-`required` blob streaming members are interpreted as
// `required`, so we can't use `member.isOptional` here.
this.members().map { symbolProvider.toSymbol(it) }.any { !it.isOptional() }
this.members().any { !symbolProvider.toSymbol(it).isOptional() && !it.hasNonNullDefault() }
}

is MapShape -> this.hasTrait<LengthTrait>()
Expand Down