Skip to content

chore: Maintenance (Deno 2, update oxc, use oxc-minifier) #7

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

Merged
merged 9 commits into from
Apr 28, 2025

Conversation

beeequeue
Copy link
Contributor

@beeequeue beeequeue commented Apr 23, 2025

I saw some stuff that could use some maintenance so I decided to do it for fun!

The diff without the lockfile is +49 -65

  • Upgraded to Deno 2
  • Upgraded oxc-transform to latest version
    • I checked the build output, and the only difference is that doc comments are retained now
    • Also added node16 as an explicit target, the minimum version in engines.node
  • Replaced terser with oxc-minify
    • It's technically experimental, but so is oxc-transform :p
    • I checked the minified sizes and they're the same and even 4 bytes smaller in walk.mjs
  • Removed pkg-up from benchmark
    • Its new name package-up is already in the benchmark

@beeequeue beeequeue changed the title Maintenance (Deno 2, update oxc, use oxc-minifier) chore: Maintenance (Deno 2, update oxc, use oxc-minifier) Apr 23, 2025
@lukeed
Copy link
Owner

lukeed commented Apr 28, 2025

Thanks! I did the Deno 2.2 upgrade myself so that the diff was simpler here. And I reverted some unwanted changes/scripts, effectively keeping the oxc-* updates. 🙇

@lukeed lukeed merged commit e7b34d9 into lukeed:main Apr 28, 2025
8 checks passed
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