Skip to content

Conversation

danleh
Copy link
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
Copy link

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.

@eqrion
Copy link
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