Skip to content

Add serialCommandExec parameter to evaluate API#6921

Open
kryptt wants to merge 1 commit intocom-lihaoyi:mainfrom
kryptt:add-serialCommandExec-to-evaluate
Open

Add serialCommandExec parameter to evaluate API#6921
kryptt wants to merge 1 commit intocom-lihaoyi:mainfrom
kryptt:add-serialCommandExec-to-evaluate

Conversation

@kryptt
Copy link
Copy Markdown
Contributor

@kryptt kryptt commented Mar 10, 2026

Summary

  • Adds a serialCommandExec: Boolean = false parameter to the evaluate method across EvaluatorApi, Evaluator, EvaluatorProxy, and EvaluatorImpl
  • Forwards the parameter to the existing execute call, allowing callers of evaluate to force serial task execution without needing to call execute directly

Test plan

  • Verified all modules compile successfully (./mill __.compile — 30195 modules)
  • Existing tests should pass unchanged since the new parameter defaults to false

🤖 Generated with Claude Code

The `evaluate` method on `EvaluatorApi`, `Evaluator`, `EvaluatorProxy`,
and `EvaluatorImpl` now accepts a `serialCommandExec` parameter
(defaulting to `false`) which is forwarded to the underlying `execute`
call. This allows callers of `evaluate` to force serial task execution
without needing to call `execute` directly.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

1 participant