knutwalker / akka-stream-json

Apache License 2.0
28 stars 14 forks source link

Bump Circe + Akka-http + Scala 2.12 Support #10

Closed mdedetrich closed 7 years ago

mdedetrich commented 7 years ago

Add support for Scala 2.12 (cross built against Scala 2.11 as well) Bumped Circe to 0.7.0 Bumped Akka-Http to 10.0.4

Note that due to the changes in Circe 0.7.0, there will be much better performance when streaming the JSON, see https://github.com/circe/circe/releases/tag/v0.7.0-M1 for more info

@knutwalker I think you need to do the version bumping yourself, let me know when you publish

codecov-io commented 7 years ago

Codecov Report

Merging #10 into master will decrease coverage by 0.28%. The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
- Coverage   80.88%   80.59%   -0.29%     
==========================================
  Files           4        4              
  Lines          68       67       -1     
  Branches        4        6       +2     
==========================================
- Hits           55       54       -1     
  Misses         13       13
Impacted Files Coverage Δ
...in/scala/de/knutwalker/akka/http/JsonSupport.scala 100% <ø> (ø) :white_check_mark:
...nutwalker/akka/http/support/CirceHttpSupport.scala 100% <ø> (ø) :white_check_mark:
...a/de/knutwalker/akka/stream/JsonStreamParser.scala 85.71% <ø> (ø) :white_check_mark:
...alker/akka/stream/support/CirceStreamSupport.scala 58.82% <33.33%> (-2.29%) :x:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update adcc330...2be84f8. Read the comment docs.

knutwalker commented 7 years ago

Thank you for taking care of this!

knutwalker commented 7 years ago

@mdedetrich I have released a version 3.2.0 for 2.12 and 2.11, the former might take a while until it's synced to maven central.

mdedetrich commented 7 years ago

@knutwalker Thanks!