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: 🐛 place nice with commonjs #1072

Closed
wants to merge 1 commit into from

Conversation

damusix
Copy link
Contributor

@damusix damusix commented Jun 23, 2024

Current TS config doesn't allow one to use normal ESM imports because they don't have a default export. This config setup should allow it to play nice with modules that setup DTS files without a default exports (like a number of Hapi modules do)

Current TS config doesn't allow one to use normal ESM imports because
they don't have a default export. This config setup should allow it to
play nice with modules that setup DTS files without a default exports
(like a number of Hapi modules do)
@damusix
Copy link
Contributor Author

damusix commented Jun 23, 2024

Fixed issue upstream. Unnecessary PR, closing.

@damusix damusix closed this Jun 23, 2024
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.

1 participant