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.

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

Restore old workflow runtime API #6344

Merged
merged 6 commits into from
Jan 28, 2025

Conversation

sfmskywalker
Copy link
Member

@sfmskywalker sfmskywalker commented Jan 28, 2025

This PR restores the previous methods on IWorkflowRuntime to allow for an easier upgrade from 3.2 to 3.3. The restored methods are marked as obsolete.


This change is Reviewable

Simplified the object creation in `CanStartWorkflowResult` by using a concise constructor format. This improves code readability and reduces verbosity without altering functionality.
Replaced direct method implementations in workflow runtimes with delegations to the new `ObsoleteWorkflowRuntime` class, simplifying the codebase. This consolidates logic and aligns the runtimes under a unified deprecated API layer.
Marked various types as obsolete, advising migration to `CreateClientAsync` methods or `IBookmarkQueue` services. Removed unused `Deprecated` namespace imports, aligning code with updated runtime standards.
@sfmskywalker sfmskywalker merged commit e2a57ac into main Jan 28, 2025
7 of 8 checks passed
@sfmskywalker sfmskywalker deleted the task/restore-previous-workflow-runtime-api branch January 28, 2025 09:37
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.

[BUG] Minor version update 3.3.0 breaks IWorkflowRuntime API. Cannot resolve symbol 'StartWorkflowAsync'
2 participants