Skip to content
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

[DM-24785] Felis yaml validator #101

Merged
merged 4 commits into from Dec 29, 2022
Merged

[DM-24785] Felis yaml validator #101

merged 4 commits into from Dec 29, 2022

Conversation

cbanek
Copy link
Contributor

@cbanek cbanek commented Dec 28, 2022

Check for some basic things, like if the table indexes are there and unique, that each column has a description, and that there are principal columns for each table.

Check for some basic things, like if the table indexes are there
and unique, that each column has a description, and that there
are principal columns for each table.
Instead of cramming it into the build action, let's put it here.
This wll make the github task go with a red X on failure.
@cbanek cbanek requested a review from frossie December 28, 2022 21:31
@cbanek cbanek merged commit 5f00d5b into main Dec 29, 2022
@cbanek cbanek deleted the tickets/DM-24785 branch December 29, 2022 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants