cucumber / cucumber-js

Cucumber for JavaScript
https://cucumber.io
MIT License
5.01k stars 1.09k forks source link

Replace usage of VError with Error.cause #2363

Closed davidjgoss closed 6 months ago

davidjgoss commented 6 months ago

🤔 What's changed?

This PR removes verror, which was used to propagate errors with the original stack trace plus local context, as it is now unmaintained and is not tolerant of thrown objects that are not instances of Error.

Instead, we can make use of a newer native JavaScript feature, Error.cause, to achieve the same thing.

⚡️ What's your motivation?

Fixes #2359

🏷️ What kind of change is this?

📋 Checklist:


This text was originally generated from a template, then edited by hand. You can modify the template here.

coveralls commented 6 months ago

Coverage Status

coverage: 98.381% (-0.002%) from 98.383% when pulling 4bf0eb2296fa2b1fe43b5797008a0def73fcba31 on fix/remove-verror into 2a95e47faeeed4bbf8ac7c8067b54ee1d18a589f on main.