Skip to content

Fix WorkflowExecutor initialization error (E0599)#1582

Open
yabhimanyu2007-sys wants to merge 2 commits intomofa-org:mainfrom
yabhimanyu2007-sys:fix-workflow-executor
Open

Fix WorkflowExecutor initialization error (E0599)#1582
yabhimanyu2007-sys wants to merge 2 commits intomofa-org:mainfrom
yabhimanyu2007-sys:fix-workflow-executor

Conversation

@yabhimanyu2007-sys
Copy link
Copy Markdown

@yabhimanyu2007-sys yabhimanyu2007-sys commented Apr 7, 2026

Summary

Fixes a compilation error caused by the use of WorkflowExecutor::default() which is not implemented.

Changes

  • Replaced WorkflowExecutor::default() with WorkflowExecutor::new(ExecutorConfig::default())

Context

The previous implementation attempted to call a non-existent default() method on WorkflowExecutor, leading to Rust error E0599.

Result

  • Project builds successfully
  • All tests pass without failures

How to Test

  1. Run cargo build
  2. Run cargo test
  3. Verify no compilation errors occur

Breaking Changes

  • None

Notes

This change aligns with the current design of WorkflowExecutor and ensures compatibility with the existing API.

@SH20RAJ
Copy link
Copy Markdown
Contributor

SH20RAJ commented Apr 7, 2026

Hey @yangrudan @lijingrs, could you please review this PR? It fixes a WorkflowExecutor initialization error (E0599) that likely affects users. Thanks! 👀

@yabhimanyu2007-sys
Copy link
Copy Markdown
Author

Reproduction

While building the project, I encountered a compilation error:

error[E0599]: no function or associated item named default found for WorkflowExecutor

This occurs where "WorkflowExecutor::default()" is used.

Fix

Replaced "WorkflowExecutor::default()" with "WorkflowExecutor::new()", which aligns with the current API.

Verification

  • "cargo build" completes successfully
  • "cargo test" passes without errors

Please let me know if this change aligns with the intended usage or if additional adjustments/tests are needed.

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