Skip to content

remove dep on fast-text-encoder #1546

Closed
@jimmywarting

Description

@jimmywarting

You only load this when needed.

"fast-text-encoding": "^1.0.0",

But i also see that you have a requirement on NodeJS v12+

"engines": {
"node": ">=12"
},

therefore this npm package could be removed, cuz text encoder/decoder is available globally since NodeJS v11

Also supporting NodeJS v12 is a bit outdated, think you should target LTS and update to something newer such as v16 that is the current LTS right now

So also think you should do:

 "engines": { 
   "node": ">=16" 
 }, 

Metadata

Metadata

Assignees

Labels

next major: breaking changethis is a change that we should wait to bundle into the next major versiontype: cleanupAn internal cleanup or hygiene concern.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions