Skip to content

Implement support for unwrapUnaryRecords option #12

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

Closed

Conversation

cfraz89
Copy link

@cfraz89 cfraz89 commented Jul 24, 2019

Respect the unwrapUnaryRecords option. When True, this exports unary records / newtypes as a type alias to the type of the single field within.

@thomasjm
Copy link
Contributor

Sorry for the long delay @cfraz89 -- I'll try to get this merged soon!

@drostron
Copy link

We recently found ourselves wanting this on a project at work. Is there still interest in this PR if we were to resolve conflicts and get CI passing?

@thomasjm
Copy link
Contributor

thomasjm commented Dec 11, 2020

Definitely! Sorry again for not getting this merged sooner. I actually had a WIP branch open where I added more tests to exercise the new functionality. There are some test failures at the moment that I haven't looked at yet.

I just pushed the branch here: https://github.com/codedownio/aeson-typescript/tree/francesco-rebased

If someone could get the tests passing I'd be happy to merge it.

@thomasjm
Copy link
Contributor

This is implemented now in 7e4c7b1

@thomasjm thomasjm closed this Aug 22, 2021
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

Successfully merging this pull request may close these issues.

3 participants