akka/akka-stream-contrib

move akka.stream.extra.Timed here

Opened this issue · 2 comments

Timed can be a good first contribution acting as a template.
https://github.com/akka/akka/tree/master/akka-stream/src/main/scala/akka/stream/extra

Note that there is a PR in progress to convert them to graph stages. That should be completed first.
akka/akka#20330

Some rewrite to make them idiomatic might also be required by this task.

@patriknw , Timed can be easily moved here but the test code https://github.com/akka/akka/blob/master/akka-stream-tests/src/test/scala/akka/stream/extra/FlowTimedSpec.scala has depedencies under the test directory of akka-stream-testkit and akka-testkit such as AkkaSpec, ScriptedTest and transitives such as ChainSetup, Coroner, akka-testkit/src/main/scala/akka/testkit/package.scala...

Any idea ?

Meanwhile, I can contribute moving, Timed and akka.stream.extra.Implicits referenced here akka/akka#20330 (comment)

We should try to rewrite the test without dependencies to the internal stuff. Heiko added a stream base class here that you can use.