Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.Comment |
commit: |
|
Hey, what will be the final Node.js version we’re going to support? I’m trying to update it in webpack-dev-server, and knowing the final supported Node.js versions would really help. Also, could we avoid increasing the requirement further and instead just make Zstd optionalsimilar to how other modules handle it, like compression does with Brotli support? |
|
Hi @bjohansebas Makes sense to make future features optional.
Don't expect any other big changes for the final v4. Hope it won't be too disruptive for the webpack-dev-server update 🙏 |
Bump minimal node 22 version to ^22.15.0 for future zstd support
related: #1106
node zstd: https://nodejs.org/api/zlib.html#zlibzstddecompressbuffer-options-callback
Summary by CodeRabbit