JounQin
JounQin
@dependabot rebase
@dependabot recreate
Hi, @nickmccurdy Do you have any idea on this? I read you may be interested on this project.
@onigoetz Thanks for showing me the impressive performance improvement! Do you think is there anything we can change to bring better performance for `synckit`? It is slower in your benchmark...
> The reason I am interested in bringing this PR to this package is the package size; sync-threads is 2-3kB, no dependencies. Whereas synckit is 1MB when installed with all...
> Though I have to admit that if possible I would also like to make a PR to remove @pkgr/utils because it brings a lot of dependencies. @onigoetz Sure, this...
@onigoetz After https://github.com/un-ts/synckit/releases/tag/v0.8.8, `synckit` is much more slimmer now! I tried to bundle it with `esbuild --minify`, the output file is about 6KB. See also https://bundlephobia.com/package/[email protected] The latest benchmark shows...
@onigoetz Awesome work again, and we can add benchmarks on CI like https://github.com/un-ts/synckit/blob/68ae0604aeefd9514bb50d61afd9e04a8f7084be/.github/workflows/ci.yml#L41-L43 It would show different benchmark results on different platforms and node versions. See also https://github.com/un-ts/synckit/actions/runs/7405983872/job/20149716081#step:6:8 ```log ┌───────────┬────────────┬──────────────┬───────────────────┬─────────────┬────────────────┬───────────┬────────────────┐...
Is there any plain for it? I need location info of AST at the same time.
That doesn't help anything. Please provide a minimal but runnable online reproduction.