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

Early return from dbt init if no available adapters #5366

Merged
merged 3 commits into from
Jun 17, 2022

Conversation

ulisesojeda
Copy link
Contributor

resolves #5365

Description

Early return from dbt init if no available adapters

Checklist

@ulisesojeda ulisesojeda requested review from a team and stu-k June 12, 2022 09:05
@cla-bot cla-bot bot added the cla:yes label Jun 12, 2022
@ulisesojeda ulisesojeda requested a review from a team as a code owner June 14, 2022 06:21
@ulisesojeda ulisesojeda requested a review from VersusFacit June 14, 2022 06:21
@jtcohen6 jtcohen6 added the ready_for_review Externally contributed PR has functional approval, ready for code review from Core engineering label Jun 14, 2022
Copy link
Contributor

@ChenyuLInx ChenyuLInx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ulisesojeda Thanks for contributing this!! The changes looks good to me. Do you think you can also adjust the tests so that they don't fail?

@ChenyuLInx ChenyuLInx added awaiting_response and removed ready_for_review Externally contributed PR has functional approval, ready for code review from Core engineering labels Jun 14, 2022
@ulisesojeda
Copy link
Contributor Author

@ulisesojeda Thanks for contributing this!! The changes looks good to me. Do you think you can also adjust the tests so that they don't fail?

Sure thing @ChenyuLInx . Could you explain me how to mock dbt.version._get_adapter_plugin_names for these tests?

@ulisesojeda ulisesojeda requested a review from ChenyuLInx June 16, 2022 17:37
Copy link
Contributor

@ChenyuLInx ChenyuLInx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! @ulisesojeda thanks for contributing and fixing the tests! And congrats on the first PR in dbt-core!!

@ChenyuLInx ChenyuLInx merged commit d257d0b into dbt-labs:main Jun 17, 2022
isidentical pushed a commit to isidentical/dbt-core that referenced this pull request Jul 7, 2022
* Exit from dbt init if no available adapters

* adding chnagie change

* fixing init_tests
agoblet pushed a commit to BigDataRepublic/dbt-core that referenced this pull request Sep 16, 2022
* Exit from dbt init if no available adapters

* adding chnagie change

* fixing init_tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CT-745] dbt init may early return if no adapters available
3 participants