Skip to content

fix(utils): Remove fixed package-type #4698

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

Closed
wants to merge 1 commit into from

Conversation

harish-prakash
Copy link

Removing the type allows consumer to choose between cjs/esm

Reasons for making this change

Using type:"module" forces the consumer to ONLY use esm. Removing the type will allow the consumer to decide between cjs/esm based on the exports configuration below.

Removing the `type` allows consumer to choose between cjs/esm
@heath-freenome
Copy link
Member

@harish-prakash If you need the CJS version of the library, you can use require instead of import, due to how we've provided the exports directive in the package.json.

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