refactor: drop slash in favor of replace all

facebook · Sep 29, 2022 · 9f80195 · 9f80195
1 parent ed3c404
commit 9f80195
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/packages/metro/package.json b/packages/metro/package.json
@@ -58,7 +58,6 @@
     "nullthrows": "^1.1.1",
     "rimraf": "^2.5.4",
     "serialize-error": "^2.1.0",
-    "slash": "^3.0.0",
     "source-map": "^0.5.6",
     "strip-ansi": "^6.0.0",
     "temp": "0.8.3",

diff --git a/packages/metro/src/lib/contextModuleTemplates.js b/packages/metro/src/lib/contextModuleTemplates.js
@@ -11,7 +11,6 @@
 
 import * as path from 'path';
 import * as os from 'os';
-import slash from 'slash';
 import type {ContextMode} from '../ModuleGraph/worker/collectDependencies';
 
 function createFileMap(
@@ -29,7 +28,7 @@ function createFileMap(
       let filePath = path.relative(modulePath, file);
 
       if (os.platform() === 'win32') {
-        filePath = slash(filePath);
+        filePath = filePath.replaceAll(path.sep, '/');
       }
 
       // NOTE(EvanBacon): I'd prefer we prevent the ability for a module to require itself (`require.context('./')`)