Linting

Author: AuHau

package.json

@@ -57,9 +57,9 @@
   "dependencies": {
     "chalk": "^2.4.2",
     "debug": "^4.1.0",
-    "datastore-fs": "^0.9.0",
-    "datastore-level": "^0.12.0",
-    "interface-datastore": "^0.7.0",
+    "datastore-fs": "~0.9.0",
+    "datastore-level": "~0.12.0",
+    "interface-datastore": "~0.7.0",
     "proper-lockfile": "^3.2.0",
     "yargs": "^12.0.5",
     "yargs-promise": "^1.1.0"

src/commands.js

@@ -40,7 +40,7 @@ async function migrate ({ repoPath, to, dry, revertOk }) {
 
   if (!to || (version <= to)) {
     await migrator.migrate(repoPath, options)
-  } else if(revertOk){
+  } else if (revertOk) {
     await migrator.revert(repoPath, to, options)
   } else {
     throw new Error('The migration would require reversion of the repo, but you have not allowed it as \'--revert-ok\' is not present.')

src/errors.js

@@ -75,4 +75,3 @@ class InvalidValueError extends Error {
 
 InvalidValueError.code = 'ERR_INVALID_VALUE'
 exports.InvalidValueError = InvalidValueError
-

src/index.js

@@ -46,7 +46,7 @@ exports.getLatestMigrationVersion = getLatestMigrationVersion
  * @param {array?} options.migrations - Array of migrations to migrate. If undefined, the bundled migrations are used. Mainly for testing purpose.
  * @returns {Promise<void>}
  */
-async function migrate (path, {toVersion, ignoreLock=false, repoOptions, onProgress, isDryRun=false, migrations}) {
+async function migrate (path, { toVersion, ignoreLock = false, repoOptions, onProgress, isDryRun = false, migrations }) {
   migrations = migrations || defaultMigrations
 
   if (!path) {
@@ -125,7 +125,7 @@ exports.migrate = migrate
  * @param {array?} options.migrations - Array of migrations to migrate. If undefined, the bundled migrations are used. Mainly for testing purpose.
  * @returns {Promise<void>}
  */
-async function revert (path, toVersion, {ignoreLock=false, repoOptions, onProgress, isDryRun=false, migrations}) {
+async function revert (path, toVersion, { ignoreLock = false, repoOptions, onProgress, isDryRun = false, migrations }) {
   migrations = migrations || defaultMigrations
 
   if (!path) {

test/index.spec.js

@@ -220,7 +220,7 @@ describe('index.js', () => {
     it('should not lock repo when ignoreLock is used', async () => {
       const options = createOptions()
       options.ignoreLock = true
-      
+
       getVersionStub.returns(4)
 
       await expect(migrator.revert('/some/path', 2, options))

test/init-test.js

@@ -37,7 +37,7 @@ module.exports = (setup, cleanup) => {
     await store.put(configKey, '')
     await store.close()
 
-    expect( await repoInit.isRepoInitialized(dir)).to.be.false()
+    expect(await repoInit.isRepoInitialized(dir)).to.be.false()
   })
 
   it('should return false with missing config key', async () => {

test/integration-test.js

@@ -13,7 +13,6 @@ const CONFIG_KEY = new Key('config')
 const VERSION_KEY = new Key('version')
 
 module.exports = (setup, cleanup) => {
-
   let dir
 
   beforeEach(async () => {

test/test-migrations/migration-2/index.js

@@ -3,7 +3,6 @@
 const Datastore = require('datastore-fs')
 const path = require('path')
 const Key = require('interface-datastore').Key
-const log = require('debug')('ipfs-repo-migrations:migration-8')
 const _set = require('just-safe-set')
 
 const CONFIG_KEY = new Key('config')
@@ -19,26 +18,26 @@ const NEW_API_ADDRESS = '/ip6/::/tcp/5001'
  */
 
 function datastoreFactory (repoPath, options) {
-  let storageBackend, storageBackendOptions
-  if (options !== undefined
-    && options['storageBackends'] !== undefined
-    && options['storageBackends']['root'] !== undefined
+  let StorageBackend, storageBackendOptions
+  if (options !== undefined &&
+    options.storageBackends !== undefined &&
+    options.storageBackends.root !== undefined
   ) {
-    storageBackend = options['storageBackends']['root']
+    StorageBackend = options.storageBackends.root
   } else {
-    storageBackend = Datastore
+    StorageBackend = Datastore
   }
 
-  if (options !== undefined
-    && options['storageBackendOptions'] !== undefined
-    && options['storageBackendOptions']['root'] !== undefined
+  if (options !== undefined &&
+    options.storageBackendOptions !== undefined &&
+    options.storageBackendOptions.root !== undefined
   ) {
-    storageBackendOptions = options['storageBackends']['root']
+    storageBackendOptions = options.storageBackendOptions.root
   } else {
-    storageBackendOptions = { 'extension': '' }
+    storageBackendOptions = { extension: '' }
   }
 
-  return new storageBackend(path.join(repoPath), storageBackendOptions)
+  return new StorageBackend(path.join(repoPath), storageBackendOptions)
 }
 
 function addNewApiAddress (config) {