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.

Sign up for GitHub

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

Jump to bottom

Add data_type to columns #75

Closed
GSokol opened this issue Aug 17, 2022 · 0 comments · Fixed by #76
Closed

Add data_type to columns #75

GSokol opened this issue Aug 17, 2022 · 0 comments · Fixed by #76
Labels
enhancement New feature or request

Comments

@GSokol
Copy link
Contributor

GSokol commented Aug 17, 2022 •
edited
Loading

Describe the feature

To make the generated documentation more informative it could be useful to add include_data_types flag to generate_source macro which leads to adding data_type property to column YAML.

Describe alternatives you've considered

That could be done manually or with another macro.

Additional context

Who will this benefit?

Anyone who got a huge legacy of the source tables to handle.

Are you interested in contributing this feature?

Yes, I have a patch, working for my Greenplum project.
@GSokol GSokol added the enhancement New feature or request label Aug 17, 2022
@GSokol GSokol mentioned this issue Aug 17, 2022
Merged
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add include_data_types flag to generate_source macro GSokol/dbt-codegen
1 participant
@GSokol