fix: Improve Validation Output, Declarative Config Consistency, and Docs #29
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Improve Validation Output, Declarative Config Consistency, and Docs
✅ Summary
This PR introduces several meaningful improvements across the validation logic and documentation:
🧪 Validation Improvements
_dq_errors
output refinedThe fail record output now contains only actual validation failures per row — removing padded
null
values for passed checks.ValidationSummary
now has a readable__str__()
A clean, human-friendly output for quick diagnostics or CLI logs:
Declarative config parsing now enforces kebab-case
Keys like
smaller-column
andgreater-column
are now properly supported — consistent with all other check parameters.📚 Documentation Enhancements
BatchDQEngine
import in exampleregex-match-check
file for improved clarity🔗 Related Commits
fix(check-runner): _dq_errors contains only validation failures
fix(check-definition): enforce kebab-case for check definition
fix(validation-summary): add string representation of validation summary
docs(sphinx): add correct import to example
docs(sphinx): renamed filename of regex-match-check
docs(sphinx): add image to section Integration Patterns
Let me know if this should be split across changelog categories or released under a patch version.