Code review comment for lp://qastaging/~u-matt-h/nova/aws-api-validation

Revision history for this message
Matthew Hooker (u-matt-h) wrote :

> Hi Matt,
>
> Just one more fix to make your code more pythonix, then it should be good:

Hi Christopher,

Thanks again for the feedback. Although my patch doesn't exhibit any instances of it, the idea is that validators could be composable. You could conceivably want to chain, for example, a url path validator and a string validator with a max length value.

« Back to merge proposal