Skip to content

Commit 842dd99

Browse files
Charles Lydingfilipesilva
Charles Lyding
authored andcommitted
refactor(@angular/cli): remove unneeded rimraf dependency
1 parent b1b1160 commit 842dd99

File tree

6 files changed

+10
-17
lines changed

6 files changed

+10
-17
lines changed

package.json

-2
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,6 @@
8181
"postcss-url": "^5.1.2",
8282
"raw-loader": "^0.5.1",
8383
"resolve": "^1.1.7",
84-
"rimraf": "^2.5.3",
8584
"rsvp": "^3.0.17",
8685
"rxjs": "^5.0.1",
8786
"sass-loader": "^6.0.3",
@@ -120,7 +119,6 @@
120119
"@types/mock-fs": "^3.6.30",
121120
"@types/node": "^6.0.36",
122121
"@types/request": "0.0.39",
123-
"@types/rimraf": "0.0.28",
124122
"@types/semver": "^5.3.30",
125123
"@types/source-map": "^0.5.0",
126124
"@types/webpack": "^2.2.15",

packages/@angular/cli/package.json

-1
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,6 @@
6666
"postcss-url": "^5.1.2",
6767
"raw-loader": "^0.5.1",
6868
"resolve": "^1.1.7",
69-
"rimraf": "^2.5.3",
7069
"rsvp": "^3.0.17",
7170
"rxjs": "^5.0.1",
7271
"sass-loader": "^6.0.3",

packages/@angular/cli/tasks/build.ts

+2-3
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import * as rimraf from 'rimraf';
1+
import * as fs from 'fs-extra';
22
import * as path from 'path';
33
import * as webpack from 'webpack';
44

@@ -8,7 +8,6 @@ import { NgCliWebpackConfig } from '../models/webpack-config';
88
import { getWebpackStatsConfig } from '../models/webpack-configs/utils';
99
import { CliConfig } from '../models/config';
1010

11-
const fs = require('fs');
1211
const Task = require('../ember-cli/lib/models/task');
1312
const SilentError = require('silent-error');
1413

@@ -27,7 +26,7 @@ export default Task.extend({
2726
throw new SilentError('An ejected project cannot use the build command anymore.');
2827
}
2928
if (runTaskOptions.deleteOutputPath) {
30-
rimraf.sync(path.resolve(this.project.root, outputPath));
29+
fs.removeSync(path.resolve(this.project.root, outputPath));
3130
}
3231

3332
const webpackConfig = new NgCliWebpackConfig(runTaskOptions, app).buildConfig();

packages/@angular/cli/tasks/serve.ts

+2-3
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
1-
import * as fs from 'fs';
1+
import * as fs from 'fs-extra';
22
import * as path from 'path';
33
import * as chalk from 'chalk';
4-
import * as rimraf from 'rimraf';
54
import * as webpack from 'webpack';
65
import * as url from 'url';
76
import { oneLine, stripIndents } from 'common-tags';
@@ -32,7 +31,7 @@ export default Task.extend({
3231
throw new SilentError('An ejected project cannot use the build command anymore.');
3332
}
3433
if (serveTaskOptions.deleteOutputPath) {
35-
rimraf.sync(path.resolve(this.project.root, outputPath));
34+
fs.removeSync(path.resolve(this.project.root, outputPath));
3635
}
3736

3837
const serveDefaults = {

tests/e2e/utils/fs.ts

+4-5
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
1-
import * as fs from 'fs';
2-
import * as rimrafPackage from 'rimraf';
1+
import * as fs from 'fs-extra';
32
import {dirname} from 'path';
43
import {stripIndents} from 'common-tags';
54

@@ -44,14 +43,14 @@ export function deleteFile(path: string) {
4443

4544
export function rimraf(path: string) {
4645
return new Promise<void>((resolve, reject) => {
47-
rimrafPackage(path, (err?: any) => {
46+
fs.remove(path, (err?: any) => {
4847
if (err) {
4948
reject(err);
5049
} else {
5150
resolve();
5251
}
53-
})
54-
}
52+
});
53+
});
5554
}
5655

5756

tools/publish/src/build.ts

+2-3
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,11 @@
11
import {Logger} from '@ngtools/logger';
2-
import * as fs from 'fs';
2+
import * as fs from 'fs-extra';
33
import * as path from 'path';
44
import {buildSchema} from './build-schema';
55

66
const denodeify = require('denodeify');
77
const glob = denodeify(require('glob'));
88
const npmRun = require('npm-run');
9-
const rimraf = denodeify(require('rimraf'));
109

1110

1211
const root = path.join(__dirname, '../../..');
@@ -65,7 +64,7 @@ export default function build(packagesToBuild: string[], _opts: any,
6564
.then(() => logger.info('Deleting dist folder...'))
6665
.then(() => {
6766
if (willBuildEverything) {
68-
return rimraf(dist);
67+
return fs.remove(dist);
6968
}
7069
})
7170
.then(() => logger.info('Creating schema.d.ts...'))

0 commit comments

Comments
 (0)