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

Bundle cjs and esm #214

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

Lonli-Lokli
Copy link

Fixes #193

Changes proposed in this pull request:
Bundle both cjs and esm making a few changes as possible.

Reviewers should focus on:
Confirming the new bundle works in both cjs and esm projects.

@Lonli-Lokli
Copy link
Author

In order to make thigs work with both Node and browser I have to add tsup.
Also it was too difficult to update and make mocha+chai work with new module so test framework has been changed to vitest.

Additionally lodash deps has been changed to ESM dependency and updated imports to make it work in ESM

@Lonli-Lokli
Copy link
Author

Also I had to disable tslint because it was complaining (prettier) about both CR/CRLF. It was again non-fixable in reasonable time. Can you consider migrating to eslint and run prettier on commit ?

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.

support ESM module
1 participant