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

Instances for Currency #4562

Open
massimosiani opened this issue Feb 19, 2024 · 7 comments
Open

Instances for Currency #4562

massimosiani opened this issue Feb 19, 2024 · 7 comments

Comments

@massimosiani
Copy link

Hi,
I often work with the java Currency type, but wasn't able to find the basic instances for that, e.g. Show and Eq.
I also looked in the mouse project.
Would them be a good fit somewhere?

@rossabaker
Copy link
Member

Every time I ask "why don't we support ____" from java.*, the answer is lacking support in Scala.js and/or Scala Native. Unless that's also the problem here, I see no reason not to support them.

@armanbilge
Copy link
Member

Lack of support on JS/Native doesn't need to stop us. We can add the instances only for JVM, or add them for all platforms but test only on JVM.

@dhinojosa
Copy link
Contributor

dhinojosa commented Feb 19, 2024 via email

@TonioGela
Copy link
Member

TonioGela commented Feb 20, 2024

Lack of support on JS/Native doesn't need to stop us. We can add the instances only for JVM, or add them for all platforms but test only on JVM.

Some times we found libraries that filled the gap, like scala-java-time in cats-time.

@armanbilge
Copy link
Member

Actually there was a proposal to move cats-time instances into Cats core.

@massimosiani
Copy link
Author

@dhinojosa thanks for pointing me to squants, I'll play with it and see if it suits in our project.

@PRAADRONZSU
Copy link

Every time I ask "why don't we support ____" from java.*, the answer is lacking support in Scala.js and/or Scala Native. Unless that's also the problem here, I see no reason not to support them.

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

6 participants