Merge pull request #123 from bcomnes/neostandard

Author: bcomnes

bin/main.js

@@ -5,9 +5,6 @@
  */
 'use strict'
 
-// TODO: remove.
-/* eslint-disable no-process-exit, no-process-env */
-
 // ------------------------------------------------------------------------------
 // Requirements
 // ------------------------------------------------------------------------------
@@ -182,5 +179,3 @@ module.exports = function main (source, outDir, args) {
     })
   }
 }
-
-/* eslint-enable no-process-exit, no-process-env */

eslint.config.js

@@ -0,0 +1,18 @@
+'use strict'
+
+const neostandard = require('neostandard')
+const mochaPlugin = require('eslint-plugin-mocha')
+
+const ignores = [
+  'coverage/**/*',
+  ...neostandard.resolveIgnoresFromGitignore()
+]
+
+module.exports = [
+  { ignores },
+  ...neostandard(),
+  {
+    ...mochaPlugin.configs.flat.recommended,
+    ignores: ['**/*', '!test/**/*'],
+  },
+]

lib/copy-sync.js

@@ -46,7 +46,7 @@ module.exports = function copySync (source, outputDir, options) {
     )
   }
 
-  options = normalizeOptions(source, outputDir, options) // eslint-disable-line no-param-reassign
+  options = normalizeOptions(source, outputDir, options)
 
   // Clean
   let cleaned = []

lib/copy.js

@@ -37,12 +37,10 @@ const removeFile = require('./utils/remove-file')
  * @returns {Promise<void>} The promise which will go fulfilled after done.
  */
 module.exports = async function copy (source, outputDir, options, callback) {
-  /* eslint-disable no-param-reassign */
   if (typeof options === 'function') {
     callback = options
     options = undefined
   }
-  /* eslint-enable no-param-reassign */
 
   assert(typeof source === 'string', "'source' should be a string.")
   assert(source.trim().length >= 1, "'source' should not be empty.")
@@ -58,7 +56,6 @@ module.exports = async function copy (source, outputDir, options, callback) {
     )
   }
 
-  // eslint-disable-next-line no-param-reassign
   options = normalizeOptions(source, outputDir, options)
 
   // Clean

lib/watch.js

@@ -44,7 +44,6 @@ module.exports = function watch (source, outputDir, options) {
     assert(typeof options === 'object', "'options' should be an object.")
   }
 
-  // eslint-disable-next-line no-param-reassign
   options = normalizeOptions(source, outputDir, options)
 
   const watcher = new Watcher(options)

package.json

@@ -16,7 +16,7 @@
     "_mocha": "_mocha \"test/*.js\" --timeout 35000",
     "clean": "rm -rf .nyc_output coverage test-ws",
     "coverage": "nyc report -r lcov && opener coverage/lcov-report/index.html",
-    "lint": "standard",
+    "lint": "eslint",
     "test": "nyc --require @babel/register npm run -s _mocha",
     "watch": "npm run -s _mocha -- --require @babel/register --watch --growl"
   },
@@ -40,14 +40,16 @@
     "@babel/register": "^7.5.5",
     "auto-changelog": "^2.2.0",
     "babel-preset-power-assert": "^3.0.0",
+    "eslint": "^9.11.1",
+    "eslint-plugin-mocha": "^10.5.0",
     "gh-release": "^7.0.2",
     "mocha": "^10.2.0",
+    "neostandard": "^0.11.6",
     "nyc": "17.0.0",
     "opener": "^1.5.1",
     "p-event": "^6.0.0",
     "power-assert": "^1.6.1",
     "shelljs": "^0.8.3",
-    "standard": "^17.1.0",
     "through": "^2.3.8"
   },
   "repository": {