-
Notifications
You must be signed in to change notification settings - Fork 92
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
Verify TypeScript Types Failure! #375
Comments
I think you should make them exportable as per your earlier PR (but as a separate PR). But while you are doing that you should remove the _ from the variable names as they are not internal. |
Open
is this issue still relevant ? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This issue is in context with #369
There are only documentation changes in the above pr still the verify typescript fails. The logs on my local are shown below.
The Linting passes on my local, but it fails on GitHub. Here are the logs
comment: The reason for verify typescript is that _state , _defaultSynth and _polySynth from Voice.tsx are imported from singer.ts but they are not exported from singer.ts
@walterbender Take a look on this and tell me how should I proceed!
The text was updated successfully, but these errors were encountered: