refactor: remove node-fetch dependency
#1102
Conversation
xeho91
requested review from
asyncapi-bot-eve,
jonaslagoni,
magicmatatjahu and
smoya
as code owners
🦋 Changeset detectedLatest commit: 7578dc3 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.
xeho91
changed the title
node-fetch dependencynode-fetch dependency
|
Note There's a chance that |
|
1 participant
Description
node-fetchand@types/node-fetchdependenciesI'm unsure what the minimum Node.js version supported for this package is. I noticed in the package root
>=18.It could create a debate whether this update is a breaking change (requires major upgrade) or not.
I personally don't think so, given that I solved only the typing issue.
Related issue(s)
Package
@types/node-fetchhas a critical vurnerable dependencyform-data: GHSA-fjxv-7rqg-78g4