Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

documentation questions, regarding use on JVM, JS and Native #464

Open
bblfish opened this issue Jul 23, 2023 · 1 comment
Open

documentation questions, regarding use on JVM, JS and Native #464

bblfish opened this issue Jul 23, 2023 · 1 comment

Comments

@bblfish
Copy link

bblfish commented Jul 23, 2023

I was re-implementing a a.pure scala version of banana-rdf, which needs xsd dateTimestamps, and then I realised I need a pure Scala implementation for scalajs. and following links I found your project.

Your documentation states:

The scala-java-time library is currently available for Scala (JVM, version 8 and later) and Scala.js (JavaScript).

Once I noticed that you use the same java.time namespace I got a bit confused, because I wondered if one was meant to override the jvm package, and how one was meant to do that. But @sjrd mentioned that the primary use-case for this lib is not for the jvm... which makes sense, but the sentence above is a bit confusing.

So I guess this is primarily useful for scala-js and Scala-native, right? (Which is actually what I was looking for)

If so that could be made a bit clearer on the documentation page. Either that or one also needs some explanation of how one would make sure one was using your java.time or the JVM one...

@bblfish bblfish changed the title documentation questions, regarging use on JVM, JS and Native documentation questions, regarding use on JVM, JS and Native Jul 23, 2023
@cquiroz
Copy link
Owner

cquiroz commented Jul 25, 2023

That's right, the main reason for the jvm side is so we can do tests comparing the results to the runtime behavior of the jwm.

In reality you only want to use it on js/native.

we should update the documentation, PRs are welcome

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants