-
-
Notifications
You must be signed in to change notification settings - Fork 621
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
feat: union generators and utils package #2289
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, let's fix tests
Some TS error 😞 |
@anshumanv Can you rebase and fix conflict here (many files was removed so it can be removed from here too) |
yep wip |
Done @alexander-akait, just need fix TS err |
Let's merge as is, I will finish this PR in the next PR (can't finish work in fork 😞 ), we should not do it anymore in future |
Closing in favour of #2343 |
What kind of change does this PR introduce?
fix
Did you add tests for your changes?
Not yet
If relevant, did you update the documentation?
Yet to
Summary
Does this PR introduce a breaking change?
Yes, deprecating a package
Other information
/cc @alexander-akait evilebott