[Feature] Add project scaffolding option #5
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
`tbd` now functions as a drop-in replacement for `dbt init` by adding features which optionally scaffold a dbt project.
This commit adds an intentional error if the target folder exists and is not empty. It also adds .gitkeep files in all the empty scaffolded folders to allow the directories to be tracked if there's an inital commit of the generated project.
Adds tests that folders are created, gitignore is matching, and dbt_project.yml is templated correctly based on ConnectionDetails.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
To make
tbda true alternative todbt init, it needs to optionally be able to scaffold a baseline dbt project. It can now do this! If you choose this option your files will be generated inside thestagingfolder of the scaffolded project.It also adds an important guard that prevents users choosing an existing, populated directory to build into. Rather than deal with how to handle overwrites, I think it's much better to just error out and ask users to either specify a new directory or choose an empty one.