gulpjs / gulpjs/remove-bom-stream
Cleanup implementation when we drop older node support
Open
- Dominant language
- JavaScript
- Stars
- 7
- Forks
- 4
- PR merge metrics
- No merged PRs in 30d
Description
We needed 2 workarounds in the implementation of this:
1. Node <12 didn't handle BOMs correctly in streaming mode
2. Before https://github.com/nodejs/node/pull/42779 - node didn't handle BOMs correctly on non-UTF8 encodings
Once we support only node versions with the fix, we should cleanup this implementation.
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.