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

Attempt at fixing Sum type schema derivation for good #1994

Merged
merged 5 commits into from
Nov 9, 2023

Conversation

kyri-petrou
Copy link
Collaborator

Fixes #1992
Fixes #1993

... and hopefully all issues with nested sum types 🥲

@kyri-petrou kyri-petrou merged commit f4eb239 into series/2.x Nov 9, 2023
10 checks passed
@kyri-petrou kyri-petrou deleted the fix-nested-sum-derivation branch November 9, 2023 04:44
@guymers
Copy link
Contributor

guymers commented Nov 9, 2023

My schema is equivalent between Scala 2 and 3 now, cheers!

Some of the unions aren't ordered alphabetically any more though.

@kyri-petrou
Copy link
Collaborator Author

kyri-petrou commented Nov 9, 2023

Hey @guymers happy to hear it's all working as expected (well, for the most of it). We just released a new version with all the fixes + some major performance improvements.

As for the ordering, if you can provide a repro and open an issue I can look into it.

And thanks again for reporting the issues, finally managed to get to the source of it and make the Scala 2 & 3 derivation equivalent (famous last words 😄 )

@kyri-petrou
Copy link
Collaborator Author

Hey @guymers, we've just discovered a regression that was introduced with these changes. Working on a fix now and we'll release a new version once I have the fix in, but in the meantime don't use version 2.4.2.

@guymers
Copy link
Contributor

guymers commented Nov 13, 2023

Only for Scala 3 yeah? I haven't merged cross version Scala 3 support yet for our GraphQL module and we wont move our app from 2.13 to 3 until after the holidays.

@kyri-petrou
Copy link
Collaborator Author

Yes only for Scala 3 👍 Should be fixed by #2005

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