search

tapjs / stack-utils

Captures and cleans stack traces.
https://www.npmjs.com/package/stack-utils
MIT License
190 stars 35 forks source link

Upgrade escape-string-regexp #52

Open TrySound opened 4 years ago

TrySound commented 4 years ago

Ref https://github.com/sindresorhus/escape-string-regexp/releases/tag/v3.0.0 and https://github.com/sindresorhus/escape-string-regexp/releases/tag/v4.0.0

TrySound commented 4 years ago

cc @isaacs

isaacs commented 3 years ago

Drops support for node 8. We'll update all deps in tap v15, leaving this aside for now. Thanks though.

TrySound commented 3 years ago

@isaacs Isn't it already dropped? Engines has node 10. https://github.com/tapjs/stack-utils/blob/master/package.json#L12-L14

coreyfarrell commented 3 years ago

@isaacs stack-utils 2.x is not used by tap v15, it is used by libtap and will be used by tap v16. I don't see an issue with this so I'm reopening.

jimmywarting commented 3 years ago

outdated - switch to v5

stianjensen commented 2 years ago

outdated - switch to v5

Created a new PR for v5: https://github.com/tapjs/stack-utils/pull/67

stianjensen commented 1 year ago

outdated - switch to v5

Although that will also require switching to ESM, so maybe bumping to 4.0 is a good first step?