Skip to content

Commit

Permalink
fix(dbt): remove local pylance error (dagster-io#16479)
Browse files Browse the repository at this point in the history
## Summary & Motivation
Pylance was misinterpreting the `\\` token. Remove it. While we're at
it, simplify some key removal logic.

## How I Tested These Changes
existing bk
  • Loading branch information
rexledesma committed Sep 13, 2023
1 parent 4eee52d commit 193b07d
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions python_modules/libraries/dagster-dbt/dagster_dbt/cli/app.py
Original file line number Diff line number Diff line change
Expand Up @@ -87,8 +87,7 @@ def copy_scaffold(
dbt_profiles_yaml: Dict[str, Any] = yaml.safe_load(dbt_profiles_path.read_bytes())

# Remove config from profiles.yml
if "config" in dbt_profiles_yaml:
dbt_profiles_yaml.pop("config", None)
dbt_profiles_yaml.pop("config", None)

dbt_adapter_packages = [
dbt_adapter_pypi_package_for_target_type(target["type"])
Expand Down Expand Up @@ -204,8 +203,8 @@ def project_scaffold_command(
Syntax(
code="\n".join(
[
f"cd '{dagster_project_dir}' \\",
" && (DAGSTER_DBT_PARSE_PROJECT_ON_LOAD=1 dagster dev)",
f"cd '{dagster_project_dir}'",
"DAGSTER_DBT_PARSE_PROJECT_ON_LOAD=1 dagster dev",
]
),
lexer="bash",
Expand Down

0 comments on commit 193b07d

Please sign in to comment.