search

circe / circe-yaml

YAML parser for circe using SnakeYAML
Apache License 2.0
141 stars 51 forks source link

Crosscompile for JS and Native with scala-yaml as the backend #431

Closed sideeffffect closed 1 month ago

sideeffffect commented 3 months ago

This is rudimentary support for YAML on JS and Native. We could add much more tests etc., and maybe there are some bugs for edge cases, but this should be fine for the beginning.

Still Scala Native 0.4.x:

armanbilge commented 3 months ago

This is rudimentary support for YAML on JS and Native.

See discussions in https://github.com/circe/circe-yaml/issues/186 and https://github.com/circe/circe-yaml/issues/285.

I have already published a library based on the scala-yaml backend and I would be happy to move it to the Circe org. I do not recommend mixing multiple backends in the same repository because that creates issues with versioning and compatibility.

https://github.com/armanbilge/circe-scala-yaml

sideeffffect commented 3 months ago

I think this is fine, making a new repository is not worth it. We can make further improvements in the scala-yaml backend after the PR is merged.

sideeffffect commented 3 months ago

Pending bugfix

sideeffffect commented 3 months ago

Now blocked on

OndrejSpanel commented 1 month ago

Great to see effort in this direction. It will be great to have a cross platform support.

How would I test 0.15.3, both on JVM and JS? The artifacts are not present in maven (yet?)

sideeffffect commented 1 month ago

The artifacts are already available: https://repo1.maven.org/maven2/io/circe/circe-yaml-scalayaml_sjs1_3/0.15.3/ https://index.scala-lang.org/circe/circe-yaml/artifacts/circe-yaml-scalayaml

Give them a spin :wink:

OndrejSpanel commented 1 month ago

I did not expect a different library name, but it makes sense - and certainly allows me testing.