-
Notifications
You must be signed in to change notification settings - Fork 40
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
Add validation for generated json metafiles #174
Add validation for generated json metafiles #174
Conversation
I think the implemtnation here is pretty solid, but I think it may be best to have an external type of validation here instead of using the same code that we're generating with. Maybe it could be as simple as piping it into JQ and ensuring it's valid json? @janosdebugs you raised the initial issue, when you mention "valid" json, do you mean that it's valid json, or valid to our format and what we expected? |
To clarify: I'm not unhappy with a Go solution, it feels more robust than shell scripting around it. That being said, I don't know the registry very well, so I'll have to rely on the more experienced colleagues for judgement here. |
Signed-off-by: Dmitry Kisler <[email protected]>
Signed-off-by: Dmitry Kisler <[email protected]>
Signed-off-by: Dmitry Kisler <[email protected]>
…ackages. Signed-off-by: Dmitry Kisler <[email protected]>
e6c065a
to
9b420b9
Compare
Signed-off-by: Dmitry Kisler <[email protected]>
Signed-off-by: Dmitry Kisler <[email protected]>
Signed-off-by: Dmitry Kisler <[email protected]>
Signed-off-by: Dmitry Kisler <[email protected]>
Signed-off-by: Dmitry Kisler <[email protected]>
Signed-off-by: Dmitry Kisler <[email protected]>
Signed-off-by: Dmitry Kisler <[email protected]>
@janosdebugs @Yantrio hey guys! The validator is ready for review. |
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.
thanks for this @kislerdm , I've added 2 tiny tiny nitpicks but overall im happy to approve.
Co-authored-by: James Humphries <[email protected]> Signed-off-by: Dmitry Kisler <[email protected]>
Co-authored-by: James Humphries <[email protected]> Signed-off-by: Dmitry Kisler <[email protected]>
@Yantrio good catch, thank you James! 😄 |
@janosdebugs Hey Janos! Could you please review the changes? Thanks! |
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.
Very nice and thorough work, thank you!
Resolves #152
What changed
Why do we need it
Examples
Provider
Module