Skip to content

Conversation

@tclose
Copy link
Contributor

@tclose tclose commented Oct 27, 2025

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Summary

Fixes a bug in the caching of workflow construction. When a workflow is initially constructed (see Workflow.construct()) with lazy inputs that are then provided in subsequent calls to the construct method, the cached version of the constructed workflow was missing the subsequently provided values

Checklist

  • I have added tests to cover my changes (if necessary)

@tclose tclose changed the base branch from subclass-or-union to main October 27, 2025 05:39
@codecov
Copy link

codecov bot commented Oct 27, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.26%. Comparing base (59bd130) to head (47cf3b0).
⚠️ Report is 13 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #847      +/-   ##
==========================================
- Coverage   88.28%   88.26%   -0.02%     
==========================================
  Files          86       86              
  Lines       17990    17994       +4     
  Branches     3509     3510       +1     
==========================================
  Hits        15883    15883              
- Misses       1729     1733       +4     
  Partials      378      378              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tclose tclose added patch Increment the patch version when merged (label from auto) release Triggers publishing with auto labels Oct 28, 2025
@tclose tclose merged commit e7926ec into main Oct 28, 2025
56 of 57 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

patch Increment the patch version when merged (label from auto) release Triggers publishing with auto

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants