v6.7.0 / 2021-04-07

Author: bjornstar

.husky/.gitignore

@@ -0,0 +1 @@
+_

.husky/pre-commit

@@ -0,0 +1,4 @@
+#!/bin/sh
+. "$(dirname "$0")/_/husky.sh"
+
+./node_modules/.bin/lint-staged

CHANGELOG.md

@@ -1,5 +1,14 @@
 # node-dev
 
+## v6.7.0 / 2021-04-07
+
+- [New Option] `--debounce` to control how long to wait before restarting
+- [New Option] `--interval` to adjust the polling interval when enabled
+- [`test`] Stop using `tap` aliases
+- [`husky`] Migrate from `v4` to `v6`
+- [dependencies] Update `semver` from `v7.3.4` to `v7.3.5`
+- [devDependencies] Update `@types/node`, `eslint`, `husky`, & `tap`
+
 ## v6.6.0 / 2021-03-23
 
 - `--clear` now clears the screen on first start

package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-dev",
-  "version": "6.6.0",
+  "version": "6.7.0",
   "description": "Restarts your app when files are modified",
   "keywords": [
     "restart",
@@ -28,7 +28,8 @@
   },
   "scripts": {
     "lint": "eslint lib test bin/node-dev",
-    "test": "node test"
+    "test": "node test",
+    "prepare": "husky install"
   },
   "dependencies": {
     "dateformat": "^3.0.3",
@@ -37,25 +38,20 @@
     "minimist": "^1.1.3",
     "node-notifier": "^8.0.1",
     "resolve": "^1.0.0",
-    "semver": "^7.3.4"
+    "semver": "^7.3.5"
   },
   "devDependencies": {
-    "@types/node": "^14.14.31",
-    "eslint": "^7.20.0",
+    "@types/node": "^14.14.37",
+    "eslint": "^7.23.0",
     "eslint-plugin-import": "^2.22.1",
-    "husky": "^4.3.8",
+    "husky": "^6.0.0",
     "lint-staged": "^10.5.4",
     "prettier": "^2.2.1",
-    "tap": "^14.11.0",
+    "tap": "^15.0.2",
     "touch": "^3.1.0",
     "ts-node": "^9.1.1",
     "typescript": "^4.1.5"
   },
-  "husky": {
-    "hooks": {
-      "pre-commit": "lint-staged"
-    }
-  },
   "lint-staged": {
     "*.js": "eslint --cache --fix",
     "*.{js,md}": "prettier --write"

test/cli.js

@@ -7,135 +7,135 @@ tap.test('notify is enabled by default', t => {
     opts: { notify }
   } = cli(['node', 'bin/node-dev', 'test']);
 
-  t.is(notify, true);
-  t.done();
+  t.equal(notify, true);
+  t.end();
 });
 
 tap.test('--no-notify', t => {
   const {
     opts: { notify }
   } = cli(['node', 'bin/node-dev', '--no-notify', 'test']);
 
-  t.is(notify, false);
-  t.done();
+  t.equal(notify, false);
+  t.end();
 });
 
 tap.test('--notify=false', t => {
   const {
     opts: { notify }
   } = cli(['node', 'bin/node-dev', '--notify=false', 'test']);
 
-  t.is(notify, false);
-  t.done();
+  t.equal(notify, false);
+  t.end();
 });
 
 tap.test('--notify', t => {
   const {
     opts: { notify }
   } = cli(['node', 'bin/node-dev', '--notify', 'test']);
 
-  t.is(notify, true);
-  t.done();
+  t.equal(notify, true);
+  t.end();
 });
 
 tap.test('--notify=true', t => {
   const {
     opts: { notify }
   } = cli(['node', 'bin/node-dev', '--notify=true', 'test']);
 
-  t.is(notify, true);
-  t.done();
+  t.equal(notify, true);
+  t.end();
 });
 
 tap.test('notify can be disabled by .node-dev.json', t => {
   const {
     opts: { notify }
   } = cli(['node', 'bin/node-dev', 'test/fixture/server.js']);
 
-  t.is(notify, false);
-  t.done();
+  t.equal(notify, false);
+  t.end();
 });
 
 tap.test('cli overrides .node-dev.json from false to true', t => {
   const {
     opts: { notify }
   } = cli(['node', 'bin/node-dev', '--notify=true', 'test/fixture/server.js']);
 
-  t.is(notify, true);
-  t.done();
+  t.equal(notify, true);
+  t.end();
 });
 
 tap.test('-r ts-node/register --inspect test/fixture/server.js', t => {
   const argv = 'node bin/node-dev -r ts-node/register --inspect test/fixture/server.js'.split(' ');
   const { nodeArgs } = cli(argv);
-  t.deepEqual(nodeArgs, ['-r', 'ts-node/register', '--inspect']);
-  t.done();
+  t.same(nodeArgs, ['-r', 'ts-node/register', '--inspect']);
+  t.end();
 });
 
 tap.test('--inspect -r ts-node/register test/fixture/server.js', t => {
   const argv = 'node bin/node-dev --inspect -r ts-node/register test/fixture/server.js'.split(' ');
   const { nodeArgs } = cli(argv);
-  t.deepEqual(nodeArgs, ['--inspect', '-r', 'ts-node/register']);
-  t.done();
+  t.same(nodeArgs, ['--inspect', '-r', 'ts-node/register']);
+  t.end();
 });
 
 tap.test('--expose_gc gc.js foo', t => {
   const argv = 'node bin/node-dev --expose_gc test/fixture/gc.js test/fixture/foo'.split(' ');
   const { nodeArgs } = cli(argv);
-  t.deepEqual(nodeArgs, ['--expose_gc']);
-  t.done();
+  t.same(nodeArgs, ['--expose_gc']);
+  t.end();
 });
 
 tap.test('clear is not enabled by default', t => {
   const {
     opts: { clear }
   } = cli(['node', 'bin/node-dev', 'test']);
 
-  t.false(clear);
-  t.done();
+  t.notOk(clear);
+  t.end();
 });
 
 tap.test('--clear enables clear', t => {
   const {
     opts: { clear }
   } = cli(['node', 'bin/node-dev', '--clear', 'test']);
 
-  t.true(clear);
-  t.done();
+  t.ok(clear);
+  t.end();
 });
 
 tap.test('interval default', t => {
   const {
     opts: { interval }
   } = cli(['node', 'bin/node-dev', 'test']);
 
-  t.is(interval, 1000);
-  t.done();
+  t.equal(interval, 1000);
+  t.end();
 });
 
 tap.test('--interval=2000', t => {
   const {
     opts: { interval }
   } = cli(['node', 'bin/node-dev', '--interval=2000', 'test']);
 
-  t.is(interval, 2000);
-  t.done();
+  t.equal(interval, 2000);
+  t.end();
 });
 
 tap.test('debounce default', t => {
   const {
     opts: { debounce }
   } = cli(['node', 'bin/node-dev', 'test']);
 
-  t.is(debounce, 10);
-  t.done();
+  t.equal(debounce, 10);
+  t.end();
 });
 
 tap.test('--debounce=2000', t => {
   const {
     opts: { debounce }
   } = cli(['node', 'bin/node-dev', '--debounce=2000', 'test']);
 
-  t.is(debounce, 2000);
-  t.done();
+  t.equal(debounce, 2000);
+  t.end();
 });

test/log.js

@@ -7,30 +7,30 @@ const noColorCfg = { ...defaultConfig, noColor: true };
 
 tap.test('log.info', t => {
   const log = logFactory(noColorCfg);
-  t.like(log.info('hello'), /\[INFO\] \d{2}:\d{2}:\d{2} hello/);
-  t.done();
+  t.match(log.info('hello'), /\[INFO\] \d{2}:\d{2}:\d{2} hello/);
+  t.end();
 });
 
 tap.test('log.warn', t => {
   const log = logFactory(noColorCfg);
-  t.like(log.warn('a warning'), /\[WARN\] \d{2}:\d{2}:\d{2} a warning/);
-  t.done();
+  t.match(log.warn('a warning'), /\[WARN\] \d{2}:\d{2}:\d{2} a warning/);
+  t.end();
 });
 
 tap.test('log.error', t => {
   const log = logFactory(noColorCfg);
-  t.like(log.error('an error'), /\[ERROR\] \d{2}:\d{2}:\d{2} an error/);
-  t.done();
+  t.match(log.error('an error'), /\[ERROR\] \d{2}:\d{2}:\d{2} an error/);
+  t.end();
 });
 
 tap.test('Disable the timestamp', t => {
   const log = logFactory({ ...noColorCfg, timestamp: false });
-  t.like(log.info('no timestamp'), /\[INFO\] no timestamp/);
-  t.done();
+  t.match(log.info('no timestamp'), /\[INFO\] no timestamp/);
+  t.end();
 });
 
 tap.test('Custom timestamp', t => {
   const log = logFactory({ ...noColorCfg, timestamp: 'yyyy-mm-dd HH:MM:ss' });
-  t.like(log.error('an error'), /\[ERROR\] \d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2} an error/);
-  t.done();
+  t.match(log.error('an error'), /\[ERROR\] \d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2} an error/);
+  t.end();
 });

test/spawn/index.js

@@ -6,7 +6,7 @@ const touchFile = require('../utils/touch-file');
 
 tap.test('should pass unknown args to node binary', t => {
   spawn('--expose_gc gc.js foo', out => {
-    t.is(out.trim(), 'foo function');
+    t.equal(out.trim(), 'foo function');
     return { exit: t.end.bind(t) };
   });
 });
@@ -79,39 +79,39 @@ tap.test('should run async code un uncaughtException handlers', t => {
 
 tap.test('should ignore caught errors', t => {
   spawn('catch-no-such-module.js', out => {
-    t.like(out, /Caught/);
+    t.match(out, /Caught/);
     return { exit: t.end.bind(t) };
   });
 });
 
 tap.test('should not show up in argv', t => {
   spawn('argv.js foo', out => {
     const argv = JSON.parse(out.replace(/'/g, '"'));
-    t.like(argv[0], /.*?node(js|\.exe)?$/);
-    t.is(argv[1], 'argv.js');
-    t.is(argv[2], 'foo');
+    t.match(argv[0], /.*?node(js|\.exe)?$/);
+    t.equal(argv[1], 'argv.js');
+    t.equal(argv[2], 'foo');
     return { exit: t.end.bind(t) };
   });
 });
 
 tap.test('should pass through the exit code', t => {
   spawn('exit.js').on('exit', code => {
-    t.is(code, 101);
+    t.equal(code, 101);
     t.end();
   });
 });
 
 tap.test('should conceal the wrapper', t => {
   // require.main should be main.js not wrap.js!
   spawn('main.js').on('exit', code => {
-    t.is(code, 0);
+    t.equal(code, 0);
     t.end();
   });
 });
 
 tap.test('should relay stdin', t => {
   const p = spawn('echo.js', out => {
-    t.is(out, 'foo');
+    t.equal(out, 'foo');
     return { exit: t.end.bind(t) };
   });
   p.stdin.write('foo');
@@ -160,7 +160,7 @@ tap.test('should kill the forked processes', t => {
 
 tap.test('should *not* set NODE_ENV', t => {
   spawn('env.js', out => {
-    t.notLike(out, /development/);
+    t.notMatch(out, /development/);
     return { exit: t.end.bind(t) };
   });
 });
@@ -211,7 +211,7 @@ tap.test('Logs timestamp by default', t => {
       touchFile('message.js');
       return out2 => {
         if (out2.match(/Restarting/)) {
-          t.like(out2, /\[INFO\] \d{2}:\d{2}:\d{2} Restarting/);
+          t.match(out2, /\[INFO\] \d{2}:\d{2}:\d{2} Restarting/);
           return { exit: t.end.bind(t) };
         }
       };
@@ -225,7 +225,7 @@ tap.test('Supports require from the command-line (ts-node/register)', t => {
       touchFile('message.js');
       return out2 => {
         if (out2.match(/Restarting/)) {
-          t.like(out2, /\[INFO\] \d{2}:\d{2}:\d{2} Restarting/);
+          t.match(out2, /\[INFO\] \d{2}:\d{2}:\d{2} Restarting/);
           return { exit: t.end.bind(t) };
         }
       };
@@ -239,7 +239,7 @@ tap.test('Uses ts-node/register for .ts files through config file (also the defa
       touchFile('message.js');
       return out2 => {
         if (out2.match(/Restarting/)) {
-          t.like(out2, /\[INFO\] \d{2}:\d{2}:\d{2} Restarting/);
+          t.match(out2, /\[INFO\] \d{2}:\d{2}:\d{2} Restarting/);
           return { exit: t.end.bind(t) };
         }
       };
@@ -253,7 +253,7 @@ tap.test('Supports ECMAScript modules', t => {
       touchFile('message.mjs');
       return out2 => {
         if (out2.match(/Restarting/)) {
-          t.like(out2, /\[INFO\] \d{2}:\d{2}:\d{2} Restarting/);
+          t.match(out2, /\[INFO\] \d{2}:\d{2}:\d{2} Restarting/);
           return { exit: t.end.bind(t) };
         }
       };
@@ -269,7 +269,7 @@ tap.test('Supports ECMAScript modules with experimental-specifier-resolution', t
       touchFile('message.js');
       return out2 => {
         if (out2.match(/Restarting/)) {
-          t.like(out2, /\[INFO\] \d{2}:\d{2}:\d{2} Restarting/);
+          t.match(out2, /\[INFO\] \d{2}:\d{2}:\d{2} Restarting/);
           return { exit: t.end.bind(t) };
         }
       };