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.

Sign up for GitHub

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

Begin warning people about spaces in model names #9886

Merged
merged 20 commits into from
Apr 12, 2024

Improve readability of logs related to problematic model names

c678cbc
Select commit
Loading
Failed to load commit list.
Refresh
Merged

Begin warning people about spaces in model names #9886

Improve readability of logs related to problematic model names
c678cbc
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Apr 12, 2024 in 1s

73.17% of diff hit (within 80.00% threshold of 88.14%)

View this Pull Request on Codecov

73.17% of diff hit (within 80.00% threshold of 88.14%)

Annotations

Check warning on line 16 in core/dbt/events/types.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/dbt/events/types.py#L16 

Added line #L16 was not covered by tests

Check warning on line 433 in core/dbt/events/types.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/dbt/events/types.py#L433 

Added line #L433 was not covered by tests

Check warning on line 435 in core/dbt/events/types.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/dbt/events/types.py#L435 

Added line #L435 was not covered by tests

Check warning on line 451 in core/dbt/events/types.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/dbt/events/types.py#L451 

Added line #L451 was not covered by tests

Check warning on line 453 in core/dbt/events/types.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/dbt/events/types.py#L453 

Added line #L453 was not covered by tests

Check warning on line 646 in core/dbt/parser/manifest.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/dbt/parser/manifest.py#L645-L646 

Added lines #L645 - L646 were not covered by tests

Check warning on line 654 in core/dbt/parser/manifest.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/dbt/parser/manifest.py#L654 

Added line #L654 was not covered by tests

Check warning on line 657 in core/dbt/parser/manifest.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/dbt/parser/manifest.py#L657 

Added line #L657 was not covered by tests

Check warning on line 667 in core/dbt/parser/manifest.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/dbt/parser/manifest.py#L666-L667 

Added lines #L666 - L667 were not covered by tests
View more details on Codecov