Skip to content
This repository was archived by the owner on Aug 11, 2022. It is now read-only.

Commit 6800f76

Browse files
committed
decompose-actions: Remove needless asynchrony
Credit: @iarna
1 parent 8f3faa3 commit 6800f76

File tree

1 file changed

+15
-20
lines changed

1 file changed

+15
-20
lines changed

‎lib/install/decompose-actions.js

Lines changed: 15 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1,33 +1,33 @@
11
'use strict'
22
var validate = require('aproba')
3-
var asyncMap = require('slide').asyncMap
43
var npm = require('../npm.js')
54

65
module.exports = function (differences, decomposed, next) {
76
validate('AAF', arguments)
8-
asyncMap(differences, function (action, done) {
7+
differences.forEach((action) => {
98
var cmd = action[0]
109
var pkg = action[1]
1110
switch (cmd) {
1211
case 'add':
13-
addSteps(decomposed, pkg, done)
12+
addSteps(decomposed, pkg)
1413
break
1514
case 'update':
16-
updateSteps(decomposed, pkg, done)
15+
updateSteps(decomposed, pkg)
1716
break
1817
case 'move':
19-
moveSteps(decomposed, pkg, done)
18+
moveSteps(decomposed, pkg)
2019
break
2120
case 'remove':
22-
removeSteps(decomposed, pkg, done)
21+
removeSteps(decomposed, pkg)
2322
break
2423
default:
25-
defaultSteps(decomposed, cmd, pkg, done)
24+
defaultSteps(decomposed, cmd, pkg)
2625
}
27-
}, next)
26+
})
27+
next()
2828
}
2929

30-
function addSteps (decomposed, pkg, done) {
30+
function addSteps (decomposed, pkg) {
3131
if (!pkg.fromBundle && !pkg.isLink) {
3232
decomposed.push(['fetch', pkg])
3333
decomposed.push(['extract', pkg])
@@ -42,31 +42,26 @@ function addSteps (decomposed, pkg, done) {
4242
decomposed.push(['finalize', pkg])
4343
}
4444
decomposed.push(['refresh-package-json', pkg])
45-
done()
4645
}
4746

48-
function updateSteps (decomposed, pkg, done) {
49-
removeSteps(decomposed, pkg.oldPkg, () => {
50-
addSteps(decomposed, pkg, done)
51-
})
47+
function updateSteps (decomposed, pkg) {
48+
removeSteps(decomposed, pkg.oldPkg)
49+
addSteps(decomposed, pkg)
5250
}
5351

54-
function removeSteps (decomposed, pkg, done) {
52+
function removeSteps (decomposed, pkg) {
5553
decomposed.push(['unbuild', pkg])
5654
decomposed.push(['remove', pkg])
57-
done()
5855
}
5956

60-
function moveSteps (decomposed, pkg, done) {
57+
function moveSteps (decomposed, pkg) {
6158
decomposed.push(['move', pkg])
6259
decomposed.push(['build', pkg])
6360
decomposed.push(['install', pkg])
6461
decomposed.push(['postinstall', pkg])
6562
decomposed.push(['refresh-package-json', pkg])
66-
done()
6763
}
6864

69-
function defaultSteps (decomposed, cmd, pkg, done) {
65+
function defaultSteps (decomposed, cmd, pkg) {
7066
decomposed.push([cmd, pkg])
71-
done()
7267
}

0 commit comments

Comments
 (0)
close