Skip to content

migrate to rslib -> tsdown#7

Merged
thinca merged 7 commits intothinca:masterfrom
ryoppippi:tsdown
Jun 13, 2025
Merged

migrate to rslib -> tsdown#7
thinca merged 7 commits intothinca:masterfrom
ryoppippi:tsdown

Conversation

@ryoppippi
Copy link
Contributor

@ryoppippi ryoppippi commented Jun 13, 2025

build result:

rslib

~/g/g/t/node-vimhelp master ❱ hyperfine -w 1 -r 5 'npm run build'
Benchmark 1: npm run build
  Time (mean ± σ):      1.449 s ±  0.008 s    [User: 2.790 s, System: 0.324 s]
  Range (min … max):    1.442 s …  1.460 s    5 runs

tsdown

~/g/g/t/node-vimhelp tsdown 16.5s ❱ hyperfine -w 1 -r 5 'npm run build'
Benchmark 1: npm run build
  Time (mean ± σ):     490.5 ms ±   5.4 ms    [User: 520.5 ms, System: 137.3 ms]
  Range (min … max):   484.1 ms … 496.0 ms    5 runs

for refernce
tsdown without publint and unused

~/g/g/t/node-vimhelp tsdown• ↓1 ❱ hyperfine -w 1 -r 5 'npm run build'
Benchmark 1: npm run build
  Time (mean ± σ):     253.3 ms ±   3.1 ms    [User: 274.1 ms, System: 75.5 ms]
  Range (min … max):   250.0 ms … 256.8 ms    5 runs

@codecov-commenter
Copy link

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

"files": [
"lib"
"lib",
"src"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

because we include sourcemap, why don't we ship src dir

Copy link
Owner

@thinca thinca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

よくわからんがたぶんヨシ!

@thinca thinca merged commit 725bad0 into thinca:master Jun 13, 2025
4 checks passed
@ryoppippi ryoppippi deleted the tsdown branch June 13, 2025 13:20
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.

3 participants