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: DTS generation #51

Merged
merged 2 commits into from
Sep 24, 2024
Merged

fix: DTS generation #51

merged 2 commits into from
Sep 24, 2024

Conversation

GeekyEggo
Copy link
Member

rollup-plugin-dts is currently unable to export class declarations as type; this results in an invalid DTS file as the DTS file includes the class, but the actual JavaScript export does not. This pull requests removes rollup-plugin-dts and flattens our types in a ./types folder that is distributed with the package. This should hopefully save us rollup headaches in the future.

@GeekyEggo GeekyEggo requested a review from ChekTek September 24, 2024 17:58
@GeekyEggo GeekyEggo merged commit 9e37a33 into main Sep 24, 2024
4 checks passed
@GeekyEggo GeekyEggo deleted the types branch September 24, 2024 21:37
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.

2 participants