Release v2.1.2 (#49)

* load npm version of punycode (specify / after require)
- dep(mocha): remove dev dep (installed by npx)
- dep(punycode): bump 2.3.0 to 2.3.1
* switch to haraka shared workflow
* ci: remove coveralls workflow (dupe)
* exclude .gitmodules from npm publish
* test: use strict assertions

Author: msimerson

.github/workflows/ci.yml

@@ -10,29 +10,14 @@ jobs:
   lint:
     uses: haraka/.github/.github/workflows/lint.yml@master
 
+  coverage:
+    uses: haraka/.github/.github/workflows/coverage.yml@master
+    secrets: inherit
+
   test:
-    needs: [ lint, get-lts ]
-    runs-on: ${{ matrix.os }}
-    strategy:
-      matrix:
-        os: [ ubuntu-latest, windows-latest ]
-        node-version: ${{ fromJson(needs.get-lts.outputs.active) }}
-      fail-fast: false
-    steps:
-    - uses: actions/checkout@v3
-    - uses: actions/setup-node@v3
-      name: Node ${{ matrix.node-version }} on ${{ matrix.os }}
-      with:
-        node-version: ${{ matrix.node-version }}
-    - run: npm install
-    - run: npm test
+    needs: [ lint ]
+    uses: haraka/.github/.github/workflows/ubuntu.yml@master
 
-  get-lts:
-     runs-on: ubuntu-latest
-     steps:
-       - id: get
-         uses: msimerson/node-lts-versions@v1
-     outputs:
-       active: ${{ steps.get.outputs.active }}
-       lts:    ${{ steps.get.outputs.lts }}
-       min:    ${{ steps.get.outputs.min }}
+  windows:
+    needs: [ lint ]
+    uses: haraka/.github/.github/workflows/windows.yml@master

.github/workflows/coveralls.yml

@@ -12,3 +12,4 @@ node_modules
 .lock-wscript
 package-lock.json
 .eslintrc.yaml
+.gitmodules

.npmignore

@@ -1,6 +1,14 @@
 
 ### Unreleased
 
+
+### [2.1.2] - 2024-02-23
+
+- load npm version of punycode (specify / after require)
+- dep(mocha): remove dev dep (installed by npx)
+- dep(punycode): bump 2.3.0 to 2.3.1
+
+
 ### 2.1.1 - 2023-08-22
 
 - feat: bundle grammar.js, build with prepack, fixes #46
@@ -77,3 +85,4 @@
 - replaced null host or user values with empty strings
 [2.1.0]: https://github.com/haraka/node-address-rfc2821/releases/tag/2.1.0
 [2.1.1]: https://github.com/haraka/node-address-rfc2821/releases/tag/2.1.1
+[2.1.2]: https://github.com/haraka/node-address-rfc2821/releases/tag/2.1.2

Changes.md

@@ -1,6 +1,6 @@
 'use strict';
 
-const punycode = require('punycode');
+const punycode = require('punycode/');
 const nearley  = require('nearley');
 
 const grammar = nearley.Grammar.fromCompiled(require('./grammar.js'));

index.js

@@ -1,6 +1,6 @@
 {
   "name": "address-rfc2821",
-  "version": "2.1.1",
+  "version": "2.1.2",
   "description": "RFC-5321 (Envelope) email address parser",
   "author": {
     "name": "The Haraka Team",
@@ -27,7 +27,6 @@
   },
   "scripts": {
     "grammar": "npx -p nearley nearleyc grammar.ne -o grammar.js",
-    "cover": "NODE_ENV=cov npx nyc --reporter=lcovonly npm run test",
     "lint": "npx eslint index.js test/*.js",
     "lintfix": "npx eslint --fix index.js test/*.js",
     "prepack": "npm run grammar",
@@ -39,13 +38,12 @@
     "url": "https://github.com/haraka/node-address-rfc2821.git"
   },
   "devDependencies": {
-    "eslint": "*",
-    "eslint-plugin-haraka": "*",
-    "mocha": "*"
+    "eslint": "8.56.0",
+    "eslint-plugin-haraka": "1.0.15"
   },
   "license": "MIT",
   "dependencies": {
     "nearley": "^2.20.1",
-    "punycode": "^2.3.0"
+    "punycode": "^2.3.1"
   }
 }

package.json

@@ -1,6 +1,6 @@
 'use strict';
 
-const assert = require('assert');
+const assert = require('node:assert/strict');
 
 const Address = require('../index').Address;
 
@@ -119,29 +119,25 @@ describe('isNull', function () {
 })
 
 describe('format()', function () {
-    it('works', function (done) {
+    it('works', function () {
         const addr = new Address('<matt@example.com>');
         assert.equal(addr.user, 'matt');
         assert.equal(addr.host, 'example.com');
         assert.equal(addr.format(), '<matt@example.com>');
-        done();
     })
 
-    it('lower cases hostnames', function (done) {
+    it('lower cases hostnames', function () {
         const addr = new Address('<matt@exAMple.com>');
         assert.equal(addr.host, 'example.com');
-        done();
     })
 
-    it('no latin escaping', function (done) {
+    it('no latin escaping', function () {
         const addr = new Address('<přílišžluťoučkýkůň@přílišžluťoučkýkůň.cz>');
         assert.equal(addr.format(), '<přílišžluťoučkýkůň@přílišžluťoučkýkůň.cz>');
-        done();
     })
 
-    it('spaces inside a quoted string', function (done) {
+    it('spaces inside a quoted string', function () {
         const addr = new Address('<"pří lišžlu ťoučkýkůň"@přílišžluťoučkýkůň.cz>');
         assert.equal(addr.format(), '<"pří lišžlu ťoučkýkůň"@přílišžluťoučkýkůň.cz>');
-        done();
     })
 })