Skip to content

Add a zlib-based compress.mjs script#172

Merged
danleh merged 5 commits into
WebKit:mainfrom
danleh:compress-js
Sep 2, 2025
Merged

Add a zlib-based compress.mjs script#172
danleh merged 5 commits into
WebKit:mainfrom
danleh:compress-js

Conversation

@danleh
Copy link
Copy Markdown
Contributor

@danleh danleh commented Sep 1, 2025

Based on compress.py from #170, with some modifications:

  • Can be run as npm run compress or simply node compress.mjs, to address @camillobruni's point about staying with a node script for consistency.
  • Uses best zlib compression setting.
  • Takes arbitrary glob patterns for input files, defaulting to all .z files for decompression.
  • Copies the file mode over to avoid spurious git diffs.

@eqrion WDYT?

Based on `compress.py` from WebKit#170, with some modifications:
- Can be run as `npm run compress` or simply `node compress.mjs`
- Uses best zlib compression setting.
- Takes arbitrary glob patterns for input files, defaulting to all .z files for decompression.
- Copies the file mode over to avoid spurious git diffs.
@danleh danleh requested a review from camillobruni September 1, 2025 23:57
@netlify
Copy link
Copy Markdown

netlify Bot commented Sep 1, 2025

Deploy Preview for webkit-jetstream-preview ready!

Name Link
🔨 Latest commit 8090c8a
🔍 Latest deploy log https://app.netlify.com/projects/webkit-jetstream-preview/deploys/68b6e5a183b059000894d4a7
😎 Deploy Preview https://deploy-preview-172--webkit-jetstream-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment thread compress.mjs Outdated
Comment thread compress.mjs Outdated
Comment thread compress.mjs Outdated
Comment thread compress.mjs Outdated
Comment thread compress.mjs Outdated
Comment thread compress.mjs Outdated
Comment thread compress.mjs Outdated
Comment thread compress.mjs Outdated
Comment thread compress.mjs Outdated
Comment thread package.json Outdated
@eqrion
Copy link
Copy Markdown
Contributor

eqrion commented Sep 2, 2025

This looks good to me, thanks!

@danleh danleh merged commit 4c0b48a into WebKit:main Sep 2, 2025
9 of 10 checks passed
@danleh danleh deleted the compress-js branch September 2, 2025 19:18
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