-
Notifications
You must be signed in to change notification settings - Fork 68
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
@errorMessage
trait
#320
@errorMessage
trait
#320
Conversation
todo: docs |
LGTM so far. |
ready to go (I think) |
modules/protocol/resources/META-INF/smithy/smithy4s.meta.smithy
Outdated
Show resolved
Hide resolved
LGTM, albeit a little thing with the selector. It seems, to me, that we should only allow |
pinging poor |
modules/protocol/resources/META-INF/smithy/smithy4s.meta.smithy
Outdated
Show resolved
Hide resolved
Added validation via a validator: structure needs a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe @Baccata when you have a minute to confirm that my requirements on the trait were ok:
- member needs to be a string
- container structure has to be annotated with
@error
No description provided.