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)

@damusix damusix force-pushed the fix/typescript-opts branch from 9056b3e to 736b2c5 Compare June 23, 2024 15:52
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 damusix force-pushed the fix/typescript-opts branch from 736b2c5 to ebb0900 Compare June 23, 2024 15:53
@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