-
Notifications
You must be signed in to change notification settings - Fork 107
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
Handle custom validate method on fakeroot #437
Comments
@wenovus Can you please create a release? Thanks! |
Created, thanks for the change. |
Re-opening for a discussion. I wonder whether we should implement this in a bit more general manner going forward -- perhaps similarly to how
In this case, we'd compare the I don't think that we need to refactor this out, but it seems like we'll end up with two APIs - and it might be a good stage to end up with a more general purpose one here in the longer-term? |
Sounds reasonable to me if there is a need for it from the community. |
@robshakir Yes that would be the right approach as now the CustomValidator is invoked only on the fakeroot. It would be great if the validator func can be invoked on a container at least. |
PR #440 is the first step for this process |
Currently the Validate method called on the Go structures which are validated against the yang schema. However, there can be use cases where one need to have a custom validation which is not the part of yang schema.
Suggestion is to have a CustomValidateOptions as ValidateOptions which can be invoked in
ytypes.Validate
.The text was updated successfully, but these errors were encountered: