-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Extended validations for the project names #5620
Conversation
Please, review this. |
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.
Thank you for the contribution! Apologies for the slow response. Just one comment that has to do with when adapters are loaded.
Co-authored-by: Stu Kilgore <stuart.kilgore@gmail.com>
@stu-k Thank you for the review. For me it does also make sense. Since we are facing the issue with getting adapter plugins during the init process. |
Looks good to me! |
Can you merge it? |
@Goodkat Merged!! Thanks for contributing this!! |
resolves #5379
Description
In response to the issue #5379
We need to check that the user-provided project name doesn't match
dbt
, a.k.a.GLOBAL_PROJECT_NAME
as well as the user-provided name also doesn't collide with one of the installed plugins' reserved package names (dbt_snowflake
,dbt_bigquery
, etc).Checklist
changie new
to create a changelog entry