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.
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
Remove ignoreUnsupportedConstraintTraits
from integration test models
#2516
Remove ignoreUnsupportedConstraintTraits
from integration test models
#2516
Conversation
Now that constraint traits are supported in server SDKs (with some corner case caveats, see #1401), we can remove `ignoreUnsupportedConstraintTraits` from the codegen config of the integration test models.
Worth implementing a check in |
A new generated diff is ready to view.
A new doc preview is ready to view. |
…-traits-from-integration-test-models
A new generated diff is ready to view.
A new doc preview is ready to view. |
Now that constraint traits are supported in server SDKs (with some corner case caveats, see #1401), this flag will almost always be useless for those early adopters of constraint traits. It is thus convenient to inform the user that they should remove it. See #2516 (comment).
…ls (#2516) * Remove `ignoreUnsupportedConstraintTraits` from integration test models Now that constraint traits are supported in server SDKs (with some corner case caveats, see #1401), we can remove `ignoreUnsupportedConstraintTraits` from the codegen config of the integration test models. * Bring back ignoreUnsupportedConstraintTraits in RestJsonValidation model
…ls (#2516) * Remove `ignoreUnsupportedConstraintTraits` from integration test models Now that constraint traits are supported in server SDKs (with some corner case caveats, see #1401), we can remove `ignoreUnsupportedConstraintTraits` from the codegen config of the integration test models. * Bring back ignoreUnsupportedConstraintTraits in RestJsonValidation model
Now that constraint traits are supported in server SDKs (with some corner case caveats, see #1401), this flag will almost always be useless for those early adopters of constraint traits. It is thus convenient to inform the user that they should remove it. See #2516 (comment). ## Checklist <!--- If a checkbox below is not applicable, then please DELETE it rather than leaving it unchecked --> - [x] I have updated `CHANGELOG.next.toml` if I made changes to the smithy-rs codegen or runtime crates ---- _By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice._
Now that constraint traits are supported in server SDKs (with some
corner case caveats, see
#1401), we can remove
ignoreUnsupportedConstraintTraits
from the codegen config of theintegration test models.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.