feat: migrate to es6 class instead of util.inherit in http2wrapper

ladjs / superagent

Ajax for Node.js and browsers (JS HTTP client). Maintained for @forwardemail, @ladjs, @spamscanner, @breejs, @cabinjs, and @lassjs.

https://ladjs.github.io/superagent/

MIT License

16.58k stars 1.33k forks source link

feat: migrate to es6 class instead of util.inherit in http2wrapper #1764

Closed tobiasdiez closed 1 year ago

tobiasdiez commented 1 year ago

It is recommended to no longer use util.inherits but instead use es6 classes and extends, see https://nodejs.org/api/util.html#utilinheritsconstructor-superconstructor. In this PR we follow this advice and migrate the http2wrapper accordingly. Other instances of util.inherit are left for follow-up PRs.

titanism commented 1 year ago

v8.1.0 released which fixes this issue, thank you

npm install superagent@8.1.0

release notes @ https://github.com/ladjs/superagent/releases/tag/v8.1.0