Skip to content

fix(web): provide type in the exports#4185

Merged
amaanq merged 1 commit intotree-sitter:masterfrom
robertohuertasm:rob/feat/fix-types
Feb 8, 2025
Merged

fix(web): provide type in the exports#4185
amaanq merged 1 commit intotree-sitter:masterfrom
robertohuertasm:rob/feat/fix-types

Conversation

@robertohuertasm
Copy link
Contributor

@robertohuertasm robertohuertasm commented Feb 5, 2025

When using TypeScript projects using other module settings than CommonJs, the types were not correctly exposed, and the compilation failed.

This adds the types path to the exports so compilation works for module: NodeNext and other variants.

I've tested this with a test project using TypeScript and several module and moduleResolution combinations.

When using TypeScript projects using other module settings than CommonJs, the types were not correctly exposed, and the compilation failed.

This adds the types path to the exports so compilation works for `module: NodeNext` and other variants.
@robertohuertasm
Copy link
Contributor Author

Related to #4187. Duplicated by #4188

@amaanq
Copy link
Member

amaanq commented Feb 8, 2025

CI failure is unrelated (and fixed by #4192), thanks!

@amaanq amaanq merged commit f95e0e3 into tree-sitter:master Feb 8, 2025
13 of 14 checks passed
@robertohuertasm robertohuertasm deleted the rob/feat/fix-types branch February 8, 2025 23:06
@clason clason added the ci:backport release-0.25 Backport label label Feb 27, 2025
@tree-sitter-ci-bot
Copy link

Successfully created backport PR for release-0.25:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants