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

Updated demos #169

Merged
merged 2 commits into from
Feb 13, 2025
Merged

Updated demos #169

merged 2 commits into from
Feb 13, 2025

Conversation

mwojtyczka
Copy link
Contributor

@mwojtyczka mwojtyczka commented Feb 12, 2025

Changes

Make the results of quality checks in demos more interesting (show multiple errors for a row).

Tests

  • manually tested
  • added unit tests
  • added integration tests

@mwojtyczka mwojtyczka requested a review from a team as a code owner February 12, 2025 12:57
@mwojtyczka mwojtyczka requested review from tombonfert and alexott and removed request for a team February 12, 2025 12:57
Copy link

github-actions bot commented Feb 12, 2025

✅ 126/126 passed, 1 skipped, 27m6s total

Running from acceptance #488

Copy link
Contributor

@pratikk-databricks pratikk-databricks left a comment

Choose a reason for hiding this comment

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

Looks good overall

@mwojtyczka mwojtyczka merged commit de11239 into main Feb 13, 2025
9 checks passed
@mwojtyczka mwojtyczka deleted the demos_improve branch February 13, 2025 15:01
mwojtyczka added a commit that referenced this pull request Feb 13, 2025
* Fixed installation process for Serverless ([#150](#150)). This commit removes the pyspark dependency from the librar to avoid spark version conflicts in Serverless and future DBR versions. CLI has been updated to install pyspark for local command execution.
* Updated demos and documentation ([#169](#169)). In this release, the quality checks in the demos have been updated to better showcase the capabilities of DQX. Documentation has been updated in various places for increase clarity. Additional contributing guides have been added.
@mwojtyczka mwojtyczka mentioned this pull request Feb 13, 2025
mwojtyczka added a commit that referenced this pull request Feb 13, 2025
* Fixed installation process for Serverless
([#150](#150)). This commit
removes the pyspark dependency from the library to avoid spark version
conflicts in Serverless and future DBR versions. CLI has been updated to
install pyspark for local command execution.
* Updated demos and documentation
([#169](#169)). In this
release, the quality checks in the demos have been updated to better
showcase the capabilities of DQX. Documentation has been updated in
various places for increased clarity. Additional contributing guides
have been added.
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