-
Notifications
You must be signed in to change notification settings - Fork 9
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
[GEN-703] Validation status with version #574
Conversation
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 doing this! LGTM! I had a few comments/questions
Quality Gate passedIssues Measures |
Trying to figure out what's wrong with the linting... (nevermind, you got it @rxu17 !) |
Quality Gate passedIssues Measures |
Problem:
Version number wasn't included in the validation status table
Solution:
Add version number. This will be a multi-process PR. We will need to either add new validation status tables, OR edit the schemas.
Extras:
Only download files that need to be validated. This should speed up the download times.
Future work:
Use fileview to get a list of files