chore: remove node-fetch

I'm not sure why we ever included this, we can just use the native fetch method. It's fine as it's a test. This will reduce the number of annoying dependency bump PRs we get.
Financial-Times · Jul 18, 2024 · 5550254 · 5550254
1 parent 05656f7
commit 5550254
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 5 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/middleware-log-errors/package.json b/packages/middleware-log-errors/package.json
@@ -21,7 +21,6 @@
   },
   "devDependencies": {
     "@financial-times/n-express": "^31.8.0",
-    "@types/express": "^4.17.21",
-    "node-fetch": "^2.7.0"
+    "@types/express": "^4.17.21"
   }
 }
diff --git a/packages/middleware-log-errors/test/end-to-end/index.spec.js b/packages/middleware-log-errors/test/end-to-end/index.spec.js
@@ -1,4 +1,3 @@
-const fetch = require('node-fetch');
 const { fork } = require('node:child_process');
 
 describe('@dotcom-reliability-kit/middleware-log-errors end-to-end', () => {