-
-
Notifications
You must be signed in to change notification settings - Fork 104
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
intellisense does not work as expected #76
Comments
If you want that in CJS, you need to use |
I see. Thanks anyway. |
@sudo-suhas Please do open a VSCode issue about this and link to this issue. It seems like something VSCode should support without having to use |
It's rather a typescript (language server) issue I guess. |
@sindresorhus I have reported the issue to VSCode - microsoft/vscode#49872. @SamVerschueren The folks at VSCode usually move the issue to the Typescript repo if it needs to be addressed there. Let us see. |
Would the change suggested in microsoft/vscode#49872 (comment) be welcome? |
Does that work? See my comment in the thread. I believe it breaks for ESM. |
When I try to use this module from VSCode in common js, intellisense does not work correctly because of default export(I think):
This is mostly a nice to have and would be good if it could be fixed.
The text was updated successfully, but these errors were encountered: