Skip to content
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

Fix issues with moduleResolution NodeNext #16

Merged
merged 3 commits into from
Apr 12, 2024

Conversation

OfficialCRUGG
Copy link
Contributor

Fixes #15

Type Definition Template now uses ./shared.js instead of ./shared to work with moduleResolution NodeNext
Updated the tests to match the newly expected updated import of ./shared as changed in commit 8a549f4
@haruaki07
Copy link
Owner

I was able to reproduce this type error, it's happened when I'm importing icons with named export.

scripts/template.js Outdated Show resolved Hide resolved
tests/template.test.js Outdated Show resolved Hide resolved
@haruaki07 haruaki07 merged commit 72b5ded into haruaki07:main Apr 12, 2024
1 check passed
@OfficialCRUGG
Copy link
Contributor Author

Sorry for not fixing this myself. Completely forgot about it. 😅
Will this be included in a release soon? It's quite essential for projects where this issue occured.

@haruaki07
Copy link
Owner

Sorry for not fixing this myself. Completely forgot about it. 😅 Will this be included in a release soon? It's quite essential for projects where this issue occured.

No worries, I just released the patch.
Feel free to open a new issue if you encounter any further problems. Thanks!

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.

Type Definition causes issues with Module Resolution NodeNext
2 participants