Skip to content

Commit 7ac0887

Browse files
authored
Remove the breakingChanges member from the required trait (#2748)
The validation logic for required is done by the `ChangedNullability` validator. Using breaking changes without context can be misleading, e.g., adding `@required` is OK as long as it's accompanied by `@clientOptional`.
1 parent 9f2c66f commit 7ac0887

1 file changed

Lines changed: 0 additions & 7 deletions

File tree

  • smithy-model/src/main/resources/software/amazon/smithy/model/loader

smithy-model/src/main/resources/software/amazon/smithy/model/loader/prelude.smithy

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -767,13 +767,6 @@ string pattern
767767
/// be present.
768768
@trait(
769769
selector: "structure > member"
770-
breakingChanges: [
771-
{
772-
change: "add"
773-
severity: "WARNING"
774-
message: "If any consumers were previously omitting this member in operation inputs, making it required is backwards incompatible"
775-
}
776-
]
777770
)
778771
structure required {}
779772

0 commit comments

Comments
 (0)