Use ES modules

Author: ehmicky

.eslintrc.js

@@ -1,3 +1,3 @@
 'use strict'
 
-module.exports = require('gulp-shared-tasks/.eslintrc')
+module.exports = require('gulp-shared-tasks/.eslintrc.modules.js')

examples/errors.js

@@ -41,6 +41,7 @@ const multipleResolves = function() {
   })
 }
 
+// eslint-disable-next-line import/no-commonjs
 module.exports = {
   uncaughtException,
   unhandledRejection,

examples/utils.sh

@@ -4,7 +4,7 @@
 dir="$(dirname "$BASH_SOURCE")"
 projectRoot="$(realpath "$dir/..")"
 binaryName="$(basename "$projectRoot")"
-pathToBinary="build/src/bin/index.js"
+pathToBinary="build/src/bin/main.js"
 
 cd "$dir"
 
@@ -16,4 +16,4 @@ alias "$binaryName"="$projectRoot/$pathToBinary"
 
 # This mocks node's `-r` flag so that examples look the same as if the library
 # directly installed.
-alias node='node -r "./utils"'
+alias node='node -r "./utils.js"'

package.json

@@ -1,7 +1,7 @@
 {
   "name": "log-process-errors",
   "version": "4.0.0",
-  "main": "build/src/index.js",
+  "main": "build/src/main.js",
   "files": [
     "build/src",
     "build/register",

register/ava.js

@@ -1,7 +1,5 @@
 // Meant to be performed as `node -r log-process-errors/build/register/ava`
-// or `require('log-process-errors/build/register/ava')`.
-'use strict'
-
-const logProcessErrors = require('../src')
+// or `import 'log-process-errors/build/register/ava'`.
+import logProcessErrors from '../src/main.js'
 
 logProcessErrors({ testing: 'ava' })

register/index.js

@@ -1,7 +1,7 @@
+/* eslint-disable filenames/no-index */
 // Meant to be performed as `node -r log-process-errors/build/register`
-// or `require('log-process-errors/build/register')`.
-'use strict'
-
-const logProcessErrors = require('../src')
+// or `import 'log-process-errors/build/register'`.
+import logProcessErrors from '../src/main.js'
 
 logProcessErrors()
+/* eslint-enable filenames/no-index */

register/jasmine.js

@@ -1,7 +1,5 @@
 // Meant to be performed as `node -r log-process-errors/build/register/jasmine`
-// or `require('log-process-errors/build/register/jasmine')`.
-'use strict'
-
-const logProcessErrors = require('../src')
+// or `import 'log-process-errors/build/register/jasmine'`.
+import logProcessErrors from '../src/main.js'
 
 logProcessErrors({ testing: 'jasmine' })

register/mocha.js

@@ -1,7 +1,5 @@
 // Meant to be performed as `node -r log-process-errors/build/register/mocha`
-// or `require('log-process-errors/build/register/mocha')`.
-'use strict'
-
-const logProcessErrors = require('../src')
+// or `import 'log-process-errors/build/register/mocha'`.
+import logProcessErrors from '../src/main.js'
 
 logProcessErrors({ testing: 'mocha' })

register/node-tap.js

@@ -2,10 +2,8 @@
 // Look for `spawn-wrap` in the codebase to see why Istanbul cannot cover this.
 /* eslint-disable filenames/match-regex, unicorn/filename-case */
 // Meant to be performed as `node -r log-process-errors/build/register/node-tap`
-// or `require('log-process-errors/build/register/node-tap')`.
-'use strict'
-
-const logProcessErrors = require('../src')
+// or `import 'log-process-errors/build/register/node-tap'`.
+import logProcessErrors from '../src/main.js'
 
 logProcessErrors({ testing: 'node-tap' })
 /* eslint-enable filenames/match-regex, unicorn/filename-case */

register/tape.js

@@ -1,7 +1,5 @@
 // Meant to be performed as `node -r log-process-errors/build/register/tape`
-// or `require('log-process-errors/build/register/tape')`.
-'use strict'
-
-const logProcessErrors = require('../src')
+// or `import 'log-process-errors/build/register/tape'`.
+import logProcessErrors from '../src/main.js'
 
 logProcessErrors({ testing: 'tape' })