Skip to content

code review feedback

Codecov / codecov/patch failed Jan 3, 2025 in 1s

0.00% of diff hit (target 75.07%)

View this Pull Request on Codecov

0.00% of diff hit (target 75.07%)

Annotations

Check warning on line 163 in metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py#L163

Added line #L163 was not covered by tests

Check warning on line 239 in metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py#L239

Added line #L239 was not covered by tests

Check warning on line 109 in metadata-ingestion/src/datahub/ingestion/source/gc/soft_deleted_entity_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/soft_deleted_entity_cleanup.py#L108-L109

Added lines #L108 - L109 were not covered by tests

Check warning on line 169 in metadata-ingestion/src/datahub/ingestion/source/gc/soft_deleted_entity_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/soft_deleted_entity_cleanup.py#L168-L169

Added lines #L168 - L169 were not covered by tests

Check warning on line 281 in metadata-ingestion/src/datahub/ingestion/source/gc/soft_deleted_entity_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/soft_deleted_entity_cleanup.py#L280-L281

Added lines #L280 - L281 were not covered by tests

Check warning on line 288 in metadata-ingestion/src/datahub/ingestion/source/gc/soft_deleted_entity_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/soft_deleted_entity_cleanup.py#L285-L288

Added lines #L285 - L288 were not covered by tests

Check warning on line 291 in metadata-ingestion/src/datahub/ingestion/source/gc/soft_deleted_entity_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/soft_deleted_entity_cleanup.py#L290-L291

Added lines #L290 - L291 were not covered by tests

Check warning on line 298 in metadata-ingestion/src/datahub/ingestion/source/gc/soft_deleted_entity_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/soft_deleted_entity_cleanup.py#L295-L298

Added lines #L295 - L298 were not covered by tests

Check warning on line 311 in metadata-ingestion/src/datahub/ingestion/source/gc/soft_deleted_entity_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/soft_deleted_entity_cleanup.py#L311

Added line #L311 was not covered by tests