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

add --project-dir flag to allow specifying project directory #1549

Merged
merged 3 commits into from
Jun 19, 2019
Merged

add --project-dir flag to allow specifying project directory #1549

merged 3 commits into from
Jun 19, 2019

Conversation

heisencoder
Copy link
Contributor

In particular, this is the directory that contains the dbt_project.yml
file and all the related project files and directories.

Without this flag, it is necessary to cd to the project directory before
running dbt.

Please note that this change does not include unit tests, so is not ready for merging yet.

See #1544

Matt Ball added 2 commits June 17, 2019 16:22
In particular, this is the directory that contains the dbt_project.yml
file and all the related project files and directories.

Without this flag, it is necessary to cd to the project directory before
running dbt.

See #1544
@beckjake
Copy link
Contributor

Looks good to me! I just kicked off tests, I'm not sure this will make it into wilt-chamberlain, but if it not I'll take care of the rebasing onto the next release branch.

@heisencoder
Copy link
Contributor Author

Sounds good! I found a stray tab character, so pushed a quick fix.

@beckjake
Copy link
Contributor

As usual github is displaying the old results here, tests did pass: https://dev.azure.com/fishtown-analytics/dbt/_build/results?buildId=244

@drewbanin
Copy link
Contributor

Thanks for your contribution @heisencoder! Merging this one - it will go out in 0.14.0 :)

@drewbanin drewbanin merged commit f834446 into dbt-labs:dev/wilt-chamberlain Jun 19, 2019
@heisencoder heisencoder deleted the feature/add-project-dir-flag branch June 19, 2019 17:33
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.

3 participants