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

Enable type generation #129

Merged
merged 4 commits into from
Mar 20, 2024
Merged

Conversation

nix6839
Copy link
Contributor

@nix6839 nix6839 commented Mar 10, 2024

Due to a TypeScript compiler bug, not importing unused types causes a type generation error, so I added commit 8ccc56e.
Revert 8ccc56e when following TypeScript issue mentioned in the commit message is fixed.

Fix #128

nix6839 and others added 4 commits March 11, 2024 03:17
Revert this commit when following TypeScript issue is fixed: microsoft/TypeScript#42873
It serves the same purpose but correctly generates types.
Tested that dependency types can be resolved in another project, even when warning in dist folder.
@joshwilsonvu joshwilsonvu merged commit 39ca88e into solidjs-community:main Mar 20, 2024
@nix6839 nix6839 deleted the type-generation branch March 20, 2024 13:19
@joshwilsonvu
Copy link
Collaborator

@nix6839 Sorry this took so long to test out, but it's published in v0.13.2! Please let me know if there's any type-related issues.

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.

Types don't exist in bundled package
2 participants