kz930 opened a new pull request, #8369:
URL: https://github.com/apache/texera/pull/8369
### What changes were proposed in this PR?
Eight lines, unrelated to any feature. File Service's test grouping reads
its fork options inside the lambda that builds the groups:
```
Test / testGrouping := (Test / definedTests).value.map { suite =>
Tests.Group(suite.name, Seq(suite), Tests.SubProcess((Test /
forkOptions).value))
}
```
sbt hoists a `.value` written there to the top of the task, so the options
were already read once rather than once per suite. The code says the other
thing, and sbt says so on every build of the repository:
```
The evaluation of `/` inside an anonymous function is prohibited.
```
Reading it once, above the loop, is what actually happens.
### Any related issues, documentation, discussions?
None. It is on its own because it is not part of anything.
### How was this PR tested?
`print WorkflowCompilingService/Test/testGrouping` no longer prints the
warning, and the module's tests run and group as before.
### Was this PR authored or co-authored using generative AI tooling?
Generated-by: Claude Code (Opus 5)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
