Skip to content
This repository was archived by the owner on Mar 10, 2020. It is now read-only.

Commit ce91c3a

Browse files
committed
fix linting, test with complete interface-core tests
1 parent 5b61744 commit ce91c3a

File tree

4 files changed

+15
-8
lines changed

4 files changed

+15
-8
lines changed

src/add-to-dagnode-transform.js

+6-1
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,14 @@ const getDagNode = require('./get-dagnode')
55

66
// transform { Hash: '...' } objects into { path: 'string', node: DAGNode }
77
module.exports = function (err, res, send, done) {
8-
if (err) return done(err)
8+
if (err) {
9+
return done(err)
10+
}
911
async.map(res, function map (entry, next) {
1012
getDagNode(send, entry.Hash, function (err, node) {
13+
if (err) {
14+
return next(err)
15+
}
1116
var obj = {
1217
path: entry.Name,
1318
node: node

src/api/add.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ module.exports = (send) => {
1010
opts = {}
1111
}
1212

13-
var good = Buffer.isBuffer(files) ||
13+
const good = Buffer.isBuffer(files) ||
1414
isStream.isReadable(files) ||
1515
Array.isArray(files)
1616

src/get-dagnode.js

+6-4
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ const bl = require('bl')
55
const async = require('async')
66

77
module.exports = function (send, hash, cb) {
8-
98
// Retrieve the object and its data in parallel, then produce a DAGNode
109
// instance using this information.
1110
async.parallel([
@@ -21,16 +20,19 @@ module.exports = function (send, hash, cb) {
2120
}],
2221

2322
function done (err, res) {
24-
if (err) return cb(err)
23+
if (err) {
24+
return cb(err)
25+
}
2526

2627
var object = res[0]
2728
var stream = res[1]
2829

2930
stream.pipe(bl(function (err, data) {
30-
if (err) return cb(err)
31+
if (err) {
32+
return cb(err)
33+
}
3134

3235
cb(err, new DAGNode(data, object.Links))
3336
}))
3437
})
35-
3638
}

src/load-commands.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -26,13 +26,13 @@ function requireCommands () {
2626
// TODO: crowding the 'files' namespace temporarily for interface-ipfs-core
2727
// compatibility, until 'files vs mfs' naming decision is resolved.
2828
cmds.files = function (send) {
29-
var files = require('./api/files')(send)
29+
const files = require('./api/files')(send)
3030
files.add = require('./api/add')(send)
3131
return files
3232
}
3333

3434
cmds.util = function (send) {
35-
var util = {
35+
const util = {
3636
addFiles: require('./api/add-files')(send),
3737
addUrl: require('./api/add-url')(send)
3838
}

0 commit comments

Comments
 (0)