-
Notifications
You must be signed in to change notification settings - Fork 595
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
scala 3: compile and test marshalling, JMH and docs subprojects #4126
Conversation
@@ -69,7 +70,7 @@ object HttpServerWithActorInteraction { | |||
implicit val timeout: Timeout = 5.seconds | |||
|
|||
// query the actor for the current auction state | |||
val bids: Future[Bids] = (auction ? GetBids).mapTo[Bids] | |||
val bids: Future[Bids] = (auction ? (replyTo => GetBids(replyTo))).mapTo[Bids] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, a bit unfortunate
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With a typed actor I'd expected being able to do val bids: Future[Bids] = action ? (GetBids(_))
(or ask
instead of ?
which makes the param list parenthesis's more obvious), without needing the mapTo
at all.
@@ -143,7 +144,7 @@ class MiscDirectivesExamplesSpec extends RoutingSpec with CompileOnlySpec { | |||
|
|||
// tests: | |||
def entityOfSize(size: Int) = | |||
HttpEntity(ContentTypes.`text/plain(UTF-8)`, "0" * size) | |||
HttpEntity(ContentTypes.`text/plain(UTF-8)`, List.fill(size)('0').mkString) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It does not look more readable...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's true. Unfortunately "0" * size
somehow picks up the *
from PathMatchers instead of from StringOps
. Since this is just a test I didn't think it's that big of a deal - do you have a nicer way to create an entity of a given size?
Three errors around parameters remaining
4f47923
to
1191ed8
Compare
@@ -69,7 +70,7 @@ object HttpServerWithActorInteraction { | |||
implicit val timeout: Timeout = 5.seconds | |||
|
|||
// query the actor for the current auction state | |||
val bids: Future[Bids] = (auction ? GetBids).mapTo[Bids] | |||
val bids: Future[Bids] = (auction ? (replyTo => GetBids(replyTo))).mapTo[Bids] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With a typed actor I'd expected being able to do val bids: Future[Bids] = action ? (GetBids(_))
(or ask
instead of ?
which makes the param list parenthesis's more obvious), without needing the mapTo
at all.
On top of #4079
Three errors remaining around creating case classes from parameters