Skip to content

Commit 644128f

Browse files
authored
Remove unneeded del development dependency (#355)
Node.js 16 is the minimum supported version as per the package.json and this version contains the `rmSync` filesystem function which can replace the `del` package usage.
1 parent 2b25ed8 commit 644128f

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,6 @@
8181
"@types/tar": "^6.1.0",
8282
"@types/varint": "^6.0.1",
8383
"@types/yargs": "^17.0.4",
84-
"del": "^6.0.0",
8584
"extract-zip": "^2.0.1",
8685
"gts": "^6.0.2",
8786
"jest": "^29.4.1",

test/sandbox.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44

55
import * as fs from 'fs';
66
import * as p from 'path';
7-
import * as del from 'del';
87

98
import {PromiseOr} from '../lib/src/utils';
109

@@ -38,7 +37,7 @@ export async function run(
3837
await test((...paths) => p.join(testDir, ...paths));
3938
} finally {
4039
if (options?.sassPathDirs) process.env.SASS_PATH = undefined;
41-
// TODO(awjin): Change this to rmSync once we drop support for Node 12.
42-
del.sync(testDir, {force: true});
40+
41+
fs.rmSync(testDir, {force: true, recursive: true, maxRetries: 3});
4342
}
4443
}

0 commit comments

Comments
 (0)