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

Emit `ArtifactWritten` event when artifacts are written #10940

Merged

QMalcolm merged 7 commits into main from qmalcolm--emit-artifact-written-events

Oct 30, 2024

+417 −309

Merged

Emit `ArtifactWritten` event when artifacts are written #10940

Add test to check that ArtifactWritten events are being emitted

Codecov / codecov/patch succeeded Oct 30, 2024 in 0s

75.00% of diff hit (within 80.00% threshold of 89.19%)

View this Pull Request on Codecov

75.00% of diff hit (within 80.00% threshold of 89.19%)

Annotations

Check warning on line 10 in core/dbt/contracts/sql.py

codecov / codecov/patch

core/dbt/contracts/sql.py#L10

Added line #L10 was not covered by tests

Check warning on line 12 in core/dbt/contracts/sql.py

codecov / codecov/patch

core/dbt/contracts/sql.py#L12

Added line #L12 was not covered by tests

Check warning on line 54 in core/dbt/contracts/sql.py

codecov / codecov/patch

core/dbt/contracts/sql.py#L54

Added line #L54 was not covered by tests

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

codecov / codecov/patch

core/dbt/parser/manifest.py#L2023

Added line #L2023 was not covered by tests

Check warning on line 431 in core/dbt/task/runnable.py

codecov / codecov/patch

core/dbt/task/runnable.py#L431

Added line #L431 was not covered by tests

View more details on Codecov