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

Skip CLI setup in tests #1650

Merged

Conversation

plypaul
Copy link
Contributor

@plypaul plypaul commented Jan 29, 2025

The setup for the dbt project should be skipped, as otherwise, devs could see unusual errors if they have a dbt project configured while running tests.

@cla-bot cla-bot bot added the cla:yes label Jan 29, 2025
@plypaul plypaul marked this pull request as ready for review January 29, 2025 23:31
Base automatically changed from p--release-dbt-metricflow-0.8.1-updates--01 to dbt-metricflow/v0.8.1 January 30, 2025 17:59
@plypaul plypaul force-pushed the p--release-dbt-metricflow-0.8.1-updates--02 branch from 8f2831f to 348be65 Compare January 30, 2025 18:01
@plypaul plypaul merged commit 4e0654b into dbt-metricflow/v0.8.1 Jan 30, 2025
9 checks passed
@plypaul plypaul deleted the p--release-dbt-metricflow-0.8.1-updates--02 branch January 30, 2025 18:06
plypaul added a commit that referenced this pull request Jan 30, 2025
The setup for the dbt project should be skipped, as otherwise, devs
could see unusual errors if they have a dbt project configured while
running tests.
plypaul added a commit that referenced this pull request Feb 6, 2025
The setup for the dbt project should be skipped, as otherwise, devs
could see unusual errors if they have a dbt project configured while
running 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.

2 participants