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 types #2184

Open
wants to merge 2 commits into
base: next
Choose a base branch
from
Open

fix types #2184

wants to merge 2 commits into from

Conversation

mifi
Copy link

@mifi mifi commented Aug 19, 2024

Goal

if not i'm getting typescript errors when importing

Design

seems to be the way it's done

Changeset

  • change export to include types, so it works with bundler

Testing

tested in a project

if not i'm getting typescript errors when importing
fix also other exports
@yousif-bugsnag
Copy link
Contributor

Hi @mifi, thanks for raising this.

This has highlighted some wider issues with the separate main and renderer exports for this package, particularly for projects using ESM.

We have some upcoming tasks scheduled to re-work our JS packages and build processes that should resolve these, so we're going to hold off on this PR in order to address these issues together.

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