-
Notifications
You must be signed in to change notification settings - Fork 2
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
Support setting profiles-dir #25
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ConstantinoSchillebeeckx: Thank you for your PR, it looks very well done! I added some minor changes. Though, the CI is causing some problems:
- Could you rebase your branch on the main branch after fetching the changes? The isort pre-commit plugin caused problems, which are solved in the latest release.
- It looks like dbt requires a
profile
attribute to be set on theArgs
. This looks like a change done by dbt unrelated to your PR. However, since you are working on adding the profiles directory as a flag, maybe you could solve this in this PR?
@ConstantinoSchillebeeckx : I updated the testing matrix. It shows that the set-up fails since dbt 1.4. |
@JCZuurmond could you kick CI off again please? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have added some comments to the tox bit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ConstantinoSchillebeeckx : LGTM! Thanks for your contribution. Will you merge the PR? I guess you can after resolving the last conversation.
@JCZuurmond branch protection or repo setup doesn't allow me to merge this - feel free to do it. |
@ConstantinoSchillebeeckx : Your changes are available in the |
Description
Allows user to set
--profiles-dir
instead of needing to set the ENVDBT_PROFILES_DIR
- setting this option is more flexible than the ENV because it can be done at the command line or as an INI option.Checklist