-
Notifications
You must be signed in to change notification settings - Fork 16
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
Add Syntax Support for Types on Macro Parameters #229
Conversation
Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #229 +/- ##
==========================================
+ Coverage 68.42% 68.89% +0.46%
==========================================
Files 52 52
Lines 3395 3433 +38
==========================================
+ Hits 2323 2365 +42
+ Misses 1072 1068 -4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Partially implements: dbt-labs/dbt-core#11090
This change customizes the jinja parser to allow for Python-style type annotations on macro parameters. It does not implement any functionality relying on the type information. That kind of functionality will be implemented in dbt-core.
Checklist
changie new
to create a changelog entry