Skip to content

BED-8746: migrate to app.defer, update OH version dependency#19

Merged
StranDutton merged 2 commits into
mainfrom
patch/BED-8746-migrate-app-defer
Jun 24, 2026
Merged

BED-8746: migrate to app.defer, update OH version dependency#19
StranDutton merged 2 commits into
mainfrom
patch/BED-8746-migrate-app-defer

Conversation

@StranDutton

@StranDutton StranDutton commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Replaces @dlt.defer with @app.defer in organization.py for the _workflow_file_contents function.

@dlt.defer functions execute lazily outside the generator iteration loop, so exceptions in them bypass the safe resource wrapper and crash the entire pipeline. @app.defer wraps them with error handling that logs the exception allows the pipeline to continue.

Changes:

  • organization.py: @dlt.defer → @app.defer
  • pyproject.toml: added openhound>=0.2.7 to runtime dependencies
  • uv.lock updated accordingly

Verified locally by starting a collection ✅

Depends on OpenHound PR #37

@StranDutton StranDutton self-assigned this Jun 24, 2026
@StranDutton StranDutton changed the title BED-8746: migrate to app.defer BED-8746: migrate to app.defer, update OH version dependency Jun 24, 2026
@StranDutton StranDutton merged commit e52cf51 into main Jun 24, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants