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.

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

Validate more Smithy constraints at runtime #9

Open
alex-chew opened this issue Mar 4, 2022 · 2 comments
Open

Validate more Smithy constraints at runtime #9

alex-chew opened this issue Mar 4, 2022 · 2 comments
Labels
general-dafny-use New functionality or clean up for broader use of this repo

Comments

@alex-chew
Copy link
Contributor

Among Smithy constraint traits, the generated .NET shims only check for the @required trait (and technically the @enum trait).

We should also add support for @range and @length at minimum.

@robin-aws
Copy link
Contributor

Note I'm not adding this to https://github.com/awslabs/polymorph/milestone/1 because the dafny-codegen plugin won't be doing any validation: it will only handle wrapping an existing AWS SDK, and the service will be responsible for validation instead.

@robin-aws robin-aws added the general-dafny-use New functionality or clean up for broader use of this repo label Mar 1, 2023
@robin-aws
Copy link
Contributor

For the polymorphing use case/CLI, I'd prioritize #185 first so we at least don't silently do the wrong thing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
general-dafny-use New functionality or clean up for broader use of this repo
Projects
None yet
Development

No branches or pull requests

2 participants