kamon-io / kamon-http4s

Kamon Integration for http4s
Apache License 2.0
53 stars 29 forks source link

Scala 3 support kamon 2.5.7 #50

Closed vaslabs closed 1 year ago

vaslabs commented 2 years ago

Summary

Notes on http4s

Test changes

Kamon testkit

Since kamon 2.3 release a change in the tests (and the way kamon is initialised in core) were causing kamon-http4s tests to yield failures, however testkit functionality was provided to initialise kamon - necessary changes were made here

Scalatest

Scalatest was also updated since the umbrella project has a scalatest version that has no scala 3 builds.

Scala 3 compiler

Suggestions

README update

Please let me know if you want me to update the README. Assuming we know the future version (e.g. 2.3.0) we can do

kamon kamon-http4s status jdk scala http4s
2.5.7+ 2.3.0 stable 8+ 2.12, 2.13 0.22.x
2.5.7+ 2.3.0 stable 8+ 2.12, 2.13, 3.x 0.23.x
2.2.7+ 2.3.0 stable 8+ 2.12, 2.13, 3.x 1.0.x
ivantopo commented 1 year ago

Hey @vaslabs, thanks for bringing this up!

Ali-Mahmood commented 1 year ago

@vaslabs are we going to merge this one day?