Merge pull request #17 from ljharb/eslint

switch to github actions; test on a matrix; run eslint in CI

.github/workflows/lint.yml

@@ -0,0 +1,18 @@
+name: 'Tests: pretest'
+
+on: [pull_request, push]
+
+permissions:  # added using https://github.com/step-security/secure-repo
+  contents: read
+
+jobs:
+  pretest:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - uses: ljharb/actions/node/install@main
+        name: 'nvm install lts/* && npm install'
+        with:
+          node-version: 'lts/*'
+      - run: npm run pretest
+

.github/workflows/tests.yml

@@ -0,0 +1,18 @@
+name: 'Tests: node.js'
+
+on: [pull_request, push]
+
+jobs:
+  tests:
+    uses: ljharb/actions/.github/workflows/node-majors.yml@main
+    with:
+      range: '>= 12'
+      command: npm run tests-only && npm run coverage
+
+  node:
+    name: 'node tests'
+    needs: [tests]
+    runs-on: ubuntu-latest
+    steps:
+      - run: 'echo tests completed'
+

lib/Subtag.js

@@ -4,8 +4,6 @@
  * @copyright Copyright (c) 2013, Matthew Caruana Galizia
  */
 
-/*jshint node:true, esversion: 6*/
-
 'use strict';
 
 var index = require('language-subtag-registry/data/json/index.json');

lib/Tag.js

@@ -4,8 +4,6 @@
  * @copyright Copyright (c) 2013, Matthew Caruana Galizia
  */
 
-/*jshint node:true, esversion: 6*/
-
 'use strict';
 
 var index = require('language-subtag-registry/data/json/index.json');

lib/index.js

@@ -4,8 +4,6 @@
  * @copyright Copyright (c) 2013, Matthew Caruana Galizia
  */
 
-/*jshint node:true, esversion: 6*/
-
 'use strict';
 
 var Tag = require('./Tag.js');

package.json

@@ -17,7 +17,10 @@
 	},
 	"license": "MIT",
 	"scripts": {
-		"test": "nyc mocha ./test/",
+		"lint": "eslint --ext=js,mjs .",
+		"tests-only": "nyc mocha ./test/",
+		"pretest": "npm run lint",
+		"test": "npm run tests-only",
 		"coverage": "nyc report --reporter=text-lcov | coveralls"
 	},
 	"keywords": [
@@ -33,8 +36,9 @@
 	},
 	"devDependencies": {
 		"coveralls": "^3.1.1",
-		"nyc": "^15.1.0",
-		"mocha": "^10.2.0"
+		"eslint": "^8.47.0",
+		"mocha": "^10.2.0",
+		"nyc": "^15.1.0"
 	},
 	"files": [
 		"/lib"
@@ -44,11 +48,20 @@
 	},
 	"eslintConfig": {
 		"parserOptions": {
-			"ecmaVersion": 13,
-			"sourceType": "module"
+			"ecmaVersion": 2022,
+			"sourceType": "script"
 		},
 		"env": {
 			"node": true
-		}
+		},
+		"overrides": [
+			{
+				"files": "test/**",
+				"globals": {
+					"suite": false,
+					"test": false
+				}
+			}
+		]
 	}
 }

test/lib/SubtagTest.js

@@ -4,9 +4,6 @@
  * @copyright Copyright (c) 2013, Matthew Caruana Galizia
  */
 
-/*jshint node:true*/
-/*global test, suite*/
-
 'use strict';
 
 var assert = require('assert');

test/lib/TagTest.js

@@ -4,9 +4,6 @@
  * @copyright Copyright (c) 2013, Matthew Caruana Galizia
  */
 
-/*jshint node:true*/
-/*global test, suite*/
-
 'use strict';
 
 var assert = require('assert');

test/lib/index-test.js

@@ -4,9 +4,6 @@
  * @copyright Copyright (c) 2013, Matthew Caruana Galizia
  */
 
-/*jshint node:true*/
-/*global test, suite*/
-
 'use strict';
 
 var assert = require('assert');