Skip to content

Conversation

hsheth2
Copy link
Contributor

@hsheth2 hsheth2 commented May 22, 2025

These have been available for a while and have had enough validation that they work well. We now feel comfortable flipping the default.

@github-actions github-actions bot added the ingestion PR or Issue related to the ingestion of metadata label May 22, 2025
Copy link

codecov bot commented May 22, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

Copy link

alwaysmeticulous bot commented May 22, 2025

🔴 Meticulous spotted visual differences in 3 of 1318 screens tested: view and approve differences detected.

Meticulous evaluated ~9 hours of user flows against your PR.

Last updated for commit e2acac9. This comment will update as new commits are pushed.

@sgomezvillamor sgomezvillamor merged commit 29ba367 into master Jul 17, 2025
59 checks passed
@sgomezvillamor sgomezvillamor deleted the use-queries-default branch July 17, 2025 07:03
sgomezvillamor added a commit that referenced this pull request Jul 17, 2025
…#13601)

Co-authored-by: Sergio Gómez Villamor <sgomezvillamor@gmail.com>
kartikey-visa pushed a commit to kartikey-visa/datahub that referenced this pull request Jul 23, 2025
…datahub-project#13601)

Co-authored-by: Sergio Gómez Villamor <sgomezvillamor@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ingestion PR or Issue related to the ingestion of metadata pending-submitter-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants