Conversation
|
Nice! I've been interested in this as well as @lolgab. Now that Native supports Scala 3 I'm hoping that more of the ecosystem will adopt, including scodec. All of that is basically blocked from here: |
|
Is it reasonable to publish a native version without support for |
|
It is reasonable. It is also reasonable to publish without Scala 3. But, we are so close! Let's do it right, and put our energy into driving this from the bottom up. For example, I just added support for shared source folder e.g. |
|
Sure thing, whatever you think is best! BTW I just pushed a fix and |
|
It looks like the upstream efforts are stalled. Do you think it's worth pushing something out without scodec-bits, Scala 3, or refined? |
|
Depends what you mean by pushing something out :) I'd say upstreams and circe are moving at roughly the same pace e.g. see #1933. |
|
Heh, good question. I think what I meant was to submit a PR like this one if I can figure out a way to simply exclude the non-native-ready parts of the code from the native build. I know it's not great practice to have the native build support less functionality than the JVM/JS builds, but it's also better than nothing. |
|
Good morning, just adding here that |
|
Cats is now published for Native on Scala 2.12, 2.13, 3. So Circe should be able to do the same. https://github.com/typelevel/cats/releases/tag/v2.8.0 Note that my build changes in #1968 will make configuring the build for Native very easy. |
|
@adampauls were you ever actually able to run the tests? I started looking into this myself and it seems that Native is missing |
|
I am on vacation at the moment but can try to do it soon! |
|
I already opened scala-native/scala-native#2725. I'm sorry, I missed the fact that it was on your checklist you made in March. Fixing this should have been # 1 priority since it affects the Circe Core module and requires a change in Scala Native itself. We will be blocked until the next Scala Native release unfortunately. |
|
For anyone who wants to play around I've published a bootleg snapshot of Circe Native. |
|
I now have a PR up at #2017. |
|
Superseded by 2017 |
Starts to address #1077, which was closed years ago but I would re-open if I could.
I thought I would put this up as proof that there isn't too much to do. The outstanding issues are:
testingNativewhereio.circe.Spaces2PrinterSuitecannot readmodules/tests/shared/src/test/resources/io/circe/spaces2-example.json(buttestingJVMcan)"scodec-bits"dependency can support native."eu.timepit" %%% "refined"can support native.I would love to see this done but don't know the project well enough to solve all these problems quickly. I thought I would put this draft PR up for discussion. Hope that's alright!