Skip to content

Conversation

43081j
Copy link
Member

@43081j 43081j commented Mar 7, 2025

Moves to being esm-only.

Unlike other tinylib projects, this one keeps tsup since we bundle dependencies still.

If we ever decide not to bundle dependencies, we should drop tsup and ship the typescript emit as-is.

Moves to being esm-only.

Unlike other tinylib projects, this one keeps `tsup` since we bundle
dependencies still.

If we ever decide not to bundle dependencies, we should drop tsup and
ship the typescript emit as-is.
@43081j 43081j requested a review from jerome-benoit March 7, 2025 16:23
Copy link

pkg-pr-new bot commented Mar 7, 2025

Open in Stackblitz

npm i https://pkg.pr.new/tinylibs/tinybench@253

commit: ffde232

Copy link

github-actions bot commented Mar 7, 2025

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
dist/index.js 10.88 KB (0%) 218 ms (0%) 119 ms (+12.29% 🔺) 337 ms
dist/index.cjs 0 B (-100% 🔽) 0 ms (-100% 🔽) 0 ms (-100% 🔽) 0 ms

@jerome-benoit jerome-benoit changed the title feat: esm-only feat!: esm-only Mar 8, 2025
@jerome-benoit
Copy link
Collaborator

Thanks, the CI action checking the size also needs CJS handling removal.

@43081j
Copy link
Member Author

43081j commented Mar 9, 2025

Any idea how we can make it happy?

It seems confused since we removed a file

@jerome-benoit
Copy link
Collaborator

Any idea how we can make it happy?

It seems confused since we removed a file

It's not a file removal, it's an action configuration cleanup. I can take over the PR since it a branch on the upstream GitHub repo.

@43081j
Copy link
Member Author

43081j commented Mar 9, 2025

Interesting. For future reference, where does that live?

Edit:

Found it!
https://github.com/tinylibs/tinybench/blob/a6e45012bddd3f37a1fcc418fc1b6da96c1cff89/.size-limit.json

Feel free to update it, otherwise I will when I'm next at my laptop 👍

@43081j
Copy link
Member Author

43081j commented Mar 9, 2025

i've updated the config 👍 still waiting on the other tinylibs repos anyway so lets coordinate releases on discord

@jerome-benoit jerome-benoit merged commit 63e90d8 into main Mar 11, 2025
15 checks passed
@jerome-benoit jerome-benoit deleted the esm-only branch March 11, 2025 15:52
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