Merge pull request #80 from ipfs/cleanup

Fix linting issues and correct cli command descriptions and log names spaces

Author: daviddias

src/cli/commands/bootstrap/add.js

@@ -3,11 +3,11 @@
 const Command = require('ronin').Command
 const IPFS = require('../../../ipfs-core')
 const debug = require('debug')
-const log = debug('cli:version')
-log.error = debug('cli:version:error')
+const log = debug('cli:bootstrap')
+log.error = debug('cli:bootstrap:error')
 
 module.exports = Command.extend({
-  desc: 'Show peers in the bootstrap list',
+  desc: 'Add peers to the bootstrap list',
 
   options: {},
 

src/cli/commands/bootstrap/list.js

@@ -3,8 +3,8 @@
 const Command = require('ronin').Command
 const IPFS = require('../../../ipfs-core')
 const debug = require('debug')
-const log = debug('cli:version')
-log.error = debug('cli:version:error')
+const log = debug('cli:bootstrap')
+log.error = debug('cli:bootstrap:error')
 
 module.exports = Command.extend({
   desc: 'Show peers in the bootstrap list',

src/cli/commands/bootstrap/rm.js

@@ -3,11 +3,11 @@
 const Command = require('ronin').Command
 const IPFS = require('../../../ipfs-core')
 const debug = require('debug')
-const log = debug('cli:version')
-log.error = debug('cli:version:error')
+const log = debug('cli:bootstrap')
+log.error = debug('cli:bootstrap:error')
 
 module.exports = Command.extend({
-  desc: 'Show peers in the bootstrap list',
+  desc: 'Removes peers from the bootstrap list',
 
   options: {},
 

src/cli/commands/commands.js

@@ -5,7 +5,7 @@ const path = require('path')
 const ronin = require('ronin')
 
 module.exports = Command.extend({
-  desc: '',
+  desc: 'List all available commands',
 
   run: (name) => {
     const cli = ronin(path.resolve(__dirname, '..'))

src/cli/commands/config.js

@@ -9,7 +9,7 @@ const log = debug('cli:config')
 log.error = debug('cli:config:error')
 
 module.exports = Command.extend({
-  desc: 'Controls configuration variables.',
+  desc: 'Get and set IPFS config values',
 
   options: {
     bool: {

src/cli/commands/config/edit.js

@@ -7,8 +7,8 @@ const temp = require('temp')
 const async = require('async')
 const IPFS = require('../../../ipfs-core')
 const debug = require('debug')
-const log = debug('cli:version')
-log.error = debug('cli:version:error')
+const log = debug('cli:config')
+log.error = debug('cli:config:error')
 
 module.exports = Command.extend({
   desc: 'Opens the config file for editing in $EDITOR',

src/cli/commands/config/replace.js

@@ -4,8 +4,8 @@ const Command = require('ronin').Command
 const IPFS = require('../../../ipfs-core')
 const debug = require('debug')
 const path = require('path')
-const log = debug('cli:version')
-log.error = debug('cli:version:error')
+const log = debug('cli:config')
+log.error = debug('cli:config:error')
 
 module.exports = Command.extend({
   desc: 'Replaces the config with <file>',

src/cli/commands/config/show.js

@@ -3,8 +3,8 @@
 const Command = require('ronin').Command
 const IPFS = require('../../../ipfs-core')
 const debug = require('debug')
-const log = debug('cli:version')
-log.error = debug('cli:version:error')
+const log = debug('cli:config')
+log.error = debug('cli:config:error')
 
 module.exports = Command.extend({
   desc: 'Outputs the content of the config file',

tests/test-http-api/test-config.js

@@ -154,13 +154,13 @@ describe('config', () => {
         const form = new FormData()
         const headers = form.getHeaders()
 
-        streamToPromise(form).then(payload => {
+        streamToPromise(form).then((payload) => {
           api.inject({
             method: 'POST',
             url: '/api/v0/config/replace',
             headers: headers,
             payload: payload
-          }, res => {
+          }, (res) => {
             expect(res.statusCode).to.equal(400)
             done()
           })
@@ -173,13 +173,13 @@ describe('config', () => {
         form.append('file', fs.createReadStream(filePath))
         const headers = form.getHeaders()
 
-        streamToPromise(form).then(payload => {
+        streamToPromise(form).then((payload) => {
           api.inject({
             method: 'POST',
             url: '/api/v0/config/replace',
             headers: headers,
             payload: payload
-          }, res => {
+          }, (res) => {
             expect(res.statusCode).to.equal(500)
             done()
           })
@@ -193,13 +193,13 @@ describe('config', () => {
         const headers = form.getHeaders()
         const expectedConfig = JSON.parse(fs.readFileSync(filePath, 'utf8'))
 
-        streamToPromise(form).then(payload => {
+        streamToPromise(form).then((payload) => {
           api.inject({
             method: 'POST',
             url: '/api/v0/config/replace',
             headers: headers,
             payload: payload
-          }, res => {
+          }, (res) => {
             expect(res.statusCode).to.equal(200)
             expect(updatedConfig()).to.deep.equal(expectedConfig)
             done()