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

Feat/117 new algo module semantic #120

Merged
merged 10 commits into from
Jul 28, 2024
Merged

Conversation

datnguye
Copy link
Owner

related #117

Description

New module to parse Semantic Entities to understand the ER

Sample command:

export DBTERD=semantic
dbterd run -s samples/jaffle-shop

OR:

dbterd run  -a semantic -s samples/jaffle-shop

TODO in next PR: implement parse_metadata from dbtcloud query

Checklist

  • I have read the contributing guide and understand what's expected of me
  • I have run this code in development and it appears to resolve the stated issue
  • This PR includes tests, or tests are not required/relevant for this PR
  • I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR

Copy link

codecov bot commented Jul 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (1855328) to head (163ff43).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #120   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           26        27    +1     
  Lines         1020      1086   +66     
=========================================
+ Hits          1020      1086   +66     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@datnguye datnguye merged commit 627833d into main Jul 28, 2024
2 checks passed
@datnguye datnguye deleted the feat/117-new-algo-module--semantic branch July 28, 2024 07:26
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.

1 participant