pjfanning commented on code in PR #150:
URL:
https://github.com/apache/incubator-pekko-http/pull/150#discussion_r1206627450
##########
docs/src/test/scala/docs/http/scaladsl/HttpClientExampleSpec.scala:
##########
@@ -318,9 +323,10 @@ class HttpClientExampleSpec extends AnyWordSpec with
Matchers with CompileOnlySp
import pekko.http.scaladsl.unmarshalling.Unmarshal
import pekko.http.scaladsl.marshallers.sprayjson.SprayJsonSupport._
import spray.json.DefaultJsonProtocol._
+ import spray.json.RootJsonFormat
case class Pet(name: String)
- implicit val petFormat = jsonFormat1(Pet)
+ implicit val petFormat: RootJsonFormat[Pet] = jsonFormat1(Pet.apply)
Review Comment:
thinking about this again, this is spray-json code so it should be covered
by spray-json tests - we probably don't really need to do extra testing in
pekko-http
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
