Skip to content

feat(data-validator): standalone validation package#562

Merged
KFilippopolitis merged 1 commit intomasterfrom
feat/data-validator-module
Mar 12, 2026
Merged

feat(data-validator): standalone validation package#562
KFilippopolitis merged 1 commit intomasterfrom
feat/data-validator-module

Conversation

@KFilippopolitis
Copy link
Collaborator

@KFilippopolitis KFilippopolitis commented Mar 5, 2026

  • add standalone data-validator CLI package with metadata and CSV validation

  • use direct DuckDB CSV querying with fused checks and line-aware diagnostics

  • enforce folder-level dataset uniqueness with SQL normalization

  • add report-all mode with text/html outputs and grouped per-CSV reporting

  • improve diagnostics (closest column suggestions, clearer enum errors, html UX polish)

  • auto-save html reports to /tmp when --output is not provided

@KFilippopolitis KFilippopolitis force-pushed the feat/data-validator-module branch 3 times, most recently from 6837ebf to 67eabc9 Compare March 6, 2026 12:38
@KFilippopolitis KFilippopolitis changed the title feat(data-validator): deliver complete standalone validation workflow feat(data-validator): standalone validation package Mar 6, 2026
Copy link
Member

@ThanKarab ThanKarab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amazing work! 👏

Please adjust the minor comments.

@KFilippopolitis KFilippopolitis force-pushed the feat/data-validator-module branch 3 times, most recently from 5369c54 to 0556c09 Compare March 11, 2026 13:09
Copy link
Member

@ThanKarab ThanKarab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job! 👏

@KFilippopolitis KFilippopolitis force-pushed the feat/data-validator-module branch 2 times, most recently from fd04229 to 7e390fd Compare March 12, 2026 07:58
@KFilippopolitis KFilippopolitis force-pushed the feat/data-validator-module branch from 7e390fd to 8bd8146 Compare March 12, 2026 08:22
@KFilippopolitis KFilippopolitis merged commit 58af4dc into master Mar 12, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants