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: introduce UnwrapOpaque type #403
Conversation
Any thoughts on the discussion in #165? |
As far as naming goes, I do not have a preference. I can amend this PR to anything you decide. Re: I just thought this discussion is ongoing for quite some time, so why not do something about it since I am also affected by the same problems at work. |
Hey, I don't want to push here or anything... if you don't want to merge this it's fine. Just kindly let me know because we need this functionality, so I might as well find/create another lib for branded types only. |
I'm fine with adding this, but it should be called |
I have renamed the new type (as well as the PR title and description) to |
Looks good. You need to add it to the readme. |
Cool, the code has been updated accordingly. |
This PR introduces a new
UnwrapOpaque
type to remove the[tag]
from Opaque values.Why is this necessary?
TS4058
error, i.e. Return type of exported function has or is using name X from external module Y but cannot be named.Fixes #165