Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

esm with tsup demonstration #2401

Merged
merged 1 commit into from
Feb 16, 2024

Conversation

Shinigami92
Copy link
Member

@Shinigami92 Shinigami92 commented Sep 17, 2023

This PR shows that with tsup + the esm migration all tests are passing

So this is the combination of

together with outputting .cjs files instead of .esm


I also tested with the playground and everything works fine 🎉

@Shinigami92 Shinigami92 self-assigned this Sep 17, 2023
@Shinigami92 Shinigami92 changed the title chore: convert to esm esm with tsup demonstration Sep 17, 2023
@codecov
Copy link

codecov bot commented Sep 17, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (convert-to-esm@ed51481). Click here to learn what that means.

Additional details and impacted files
@@                Coverage Diff                @@
##             convert-to-esm    #2401   +/-   ##
=================================================
  Coverage                  ?   99.55%           
=================================================
  Files                     ?     2817           
  Lines                     ?   251199           
  Branches                  ?     1124           
=================================================
  Hits                      ?   250088           
  Misses                    ?     1082           
  Partials                  ?       29           

@Shinigami92 Shinigami92 added the do NOT merge yet Do not merge this PR into the target branch yet label Sep 20, 2023
@Shinigami92 Shinigami92 force-pushed the esm-with-tsup-that-supports-cjs branch from a021a8f to fbefff7 Compare September 20, 2023 15:15
@Shinigami92 Shinigami92 force-pushed the esm-with-tsup-that-supports-cjs branch from fbefff7 to 3a00c1f Compare January 19, 2024 10:15
@Shinigami92 Shinigami92 force-pushed the esm-with-tsup-that-supports-cjs branch from 3a00c1f to 1f151da Compare February 16, 2024 16:35
@Shinigami92 Shinigami92 marked this pull request as ready for review February 16, 2024 16:38
@Shinigami92 Shinigami92 requested a review from a team as a code owner February 16, 2024 16:38
@Shinigami92 Shinigami92 merged commit 0a680b0 into convert-to-esm Feb 16, 2024
12 of 16 checks passed
@Shinigami92 Shinigami92 deleted the esm-with-tsup-that-supports-cjs branch February 16, 2024 16:38
@ST-DDT ST-DDT removed the do NOT merge yet Do not merge this PR into the target branch yet label Feb 16, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants