Use ESM

Author: wooorm

.gitignore

@@ -1,8 +1,5 @@
-.nyc_output/
+*.log
 coverage/
 node_modules/
-*.log
 .DS_Store
-array-iterate.js
-array-iterate.min.js
 yarn.lock

.prettierignore

@@ -1,5 +1,2 @@
 coverage/
-*.json
 *.md
-array-iterate.js
-array-iterate.min.js

index.js

@@ -1,10 +1,6 @@
-'use strict'
-
-module.exports = iterate
-
 var own = {}.hasOwnProperty
 
-function iterate(values, callback, context) {
+export function arrayIterate(values, callback, context) {
   var index = -1
   var result
 

package.json

@@ -19,27 +19,25 @@
   "contributors": [
     "Titus Wormer <tituswormer@gmail.com> (https://wooorm.com)"
   ],
+  "sideEffects": false,
+  "type": "module",
+  "main": "index.js",
   "files": [
     "index.js"
   ],
   "devDependencies": {
-    "browserify": "^17.0.0",
-    "nyc": "^15.0.0",
+    "c8": "^7.0.0",
     "prettier": "^2.0.0",
     "remark-cli": "^9.0.0",
     "remark-preset-wooorm": "^8.0.0",
     "tape": "^5.0.0",
-    "tinyify": "^3.0.0",
     "xo": "^0.38.0"
   },
   "scripts": {
     "format": "remark . -qfo && prettier . -w --loglevel warn && xo --fix",
-    "build-bundle": "browserify . -s arrayIterate -o array-iterate.js",
-    "build-mangle": "browserify . -s arrayIterate -p tinyify -o array-iterate.min.js",
-    "build": "npm run build-bundle && npm run build-mangle",
     "test-api": "node test",
-    "test-coverage": "nyc --reporter lcov tape test.js",
-    "test": "npm run format && npm run build && npm run test-coverage"
+    "test-coverage": "c8 --check-coverage --branches 100 --functions 100 --lines 100 --statements 100 --reporter lcov node test.js",
+    "test": "npm run format && npm run test-coverage"
   },
   "prettier": {
     "tabWidth": 2,
@@ -51,8 +49,9 @@
   },
   "xo": {
     "prettier": true,
-    "esnext": false,
     "rules": {
+      "no-var": "off",
+      "prefer-arrow-callback": "off",
       "unicorn/no-this-assignment": "off",
       "unicorn/prefer-type-error": "off"
     }
@@ -61,11 +60,5 @@
     "plugins": [
       "preset-wooorm"
     ]
-  },
-  "nyc": {
-    "check-coverage": true,
-    "lines": 100,
-    "functions": 100,
-    "branches": 100
   }
 }

readme.md

@@ -9,6 +9,9 @@
 
 ## Install
 
+This package is ESM only: Node 12+ is needed to use it and it must be `import`ed
+instead of `require`d.
+
 [npm][]:
 
 ```sh
@@ -18,12 +21,12 @@ npm install array-iterate
 ## Use
 
 ```js
-var iterate = require('array-iterate')
+import {arrayIterate} from 'array-iterate'
 
 var isFirst = true
 var context = {hello: 'world'}
 
-iterate([1, 2, 3, 4], callback, context)
+arrayIterate([1, 2, 3, 4], callback, context)
 
 function callback(value, index, values) {
   console.log(this, value, index, values)
@@ -50,7 +53,10 @@ Yields:
 
 ## API
 
-### `iterate(values, callback[, context])`
+This package exports the following identifiers: `arrayIterate`.
+There is no default export.
+
+### `arrayIterate(values, callback[, context])`
 
 Works just like [`Array#forEach()`][foreach], but when `callback` returns a
 `number`, iterates over the item at `number` next.

test.js

@@ -1,28 +1,26 @@
-'use strict'
+import test from 'tape'
+import {arrayIterate} from './index.js'
 
-var test = require('tape')
-var iterate = require('.')
-
-test('iterate()', function (t) {
+test('arrayIterate()', function (t) {
   t.throws(
     function () {
-      iterate()
+      arrayIterate()
     },
     /^Error: Iterate requires that \|this\| not be undefined$/,
     'should throw without `values`'
   )
 
   t.throws(
     function () {
-      iterate({})
+      arrayIterate({})
     },
     /Error: Iterate requires that \|this\| has a `length`/,
     'should throw without `values.length`'
   )
 
   t.throws(
     function () {
-      iterate([])
+      arrayIterate([])
     },
     /^Error: `callback` must be a function$/,
     'should throw without `callback`'
@@ -32,7 +30,7 @@ test('iterate()', function (t) {
     var list = [0, 1, 2]
     var n = 0
 
-    iterate(list, function (value, index, values) {
+    arrayIterate(list, function (value, index, values) {
       st.equal(value, n)
       st.equal(index, n)
       st.equal(values, list)
@@ -49,7 +47,7 @@ test('iterate()', function (t) {
     var self = this
     var n = 0
 
-    iterate(
+    arrayIterate(
       [1, 2, 3],
       function () {
         st.equal(this, self)
@@ -65,7 +63,7 @@ test('iterate()', function (t) {
   t.test('should use the given return value', function (st) {
     var n = 0
 
-    iterate([0, 1, 2], function (value, index) {
+    arrayIterate([0, 1, 2], function (value, index) {
       n++
 
       st.equal(value, index)
@@ -89,7 +87,7 @@ test('iterate()', function (t) {
 
     list.push(magicNumber + 1)
 
-    iterate(list, function (value, index) {
+    arrayIterate(list, function (value, index) {
       st.equal(value, magicNumber + 1)
       st.equal(index, magicNumber)
       n = index
@@ -104,7 +102,7 @@ test('iterate()', function (t) {
     var n = 0
     var results = ['a', 'b', 'a', 'b', 'c', 'd']
 
-    iterate(['a', 'b', 'c', 'd'], function (value) {
+    arrayIterate(['a', 'b', 'c', 'd'], function (value) {
       st.equal(value, results[n])
       n++