Make tests run with skips on node 18

README.md

@@ -45,8 +45,8 @@ console.log(trace("123"));
 ## Notes:
 
 - This currently is only tested on Node 18+, no browser version yet.
-- `--experimental-vm-modules` is required for the async versions that
-  allow importing libraries.
+- Node 20.8+ and `--experimental-vm-modules` are required for the async
+  versions that allow importing libraries.
 - This is for NON-performance-sensitive code (e.g. prototypes), because the
   parser with be generated every time the template is evaluated.
 - If your parse function's variable name has exactly five letters (like

eslint.config.js

@@ -1,4 +1,5 @@
 import mod from "@peggyjs/eslint-config/flat/module.js";
+import modern from "@peggyjs/eslint-config/flat/modern.js";
 import peggyjs from "@peggyjs/eslint-plugin/lib/flat/recommended.js";
 
 export default [
@@ -11,4 +12,10 @@ export default [
   },
   mod,
   peggyjs,
+  {
+    files: [
+      "test/**",
+    ],
+    modern,
+  },
 ];

package.json

@@ -40,6 +40,7 @@
     "@peggyjs/eslint-plugin": "2.0.0",
     "@types/node": "20.11.20",
     "c8": "9.1.0",
+    "semver": "7.6.0",
     "typescript": "5.3.3"
   },
   "packageManager": "[email protected]",

test/index-spec.js

@@ -1,5 +1,6 @@
 import { default as peggy, withImports } from "../lib/index.js";
 import assert from "node:assert";
+import semver from "semver";
 import test from "node:test";
 
 // Don't syntax-highlight the intentionally-invalid code.
@@ -15,15 +16,15 @@ test("bad grammar", () => {
     () => bad`foo = "foo`,
     // The position of the dquote in the line above.  This will change
     // if the line above moves in the file.
-    /index-spec\.js:15:21\n   \|\n15 \| foo = "foo\n   \|       \^/
+    /index-spec\.js:16:21\n   \|\n16 \| foo = "foo\n   \|       \^/
   );
 });
 
 test("bad input", () => {
   const parser = peggy`foo = "foo"`;
   assert.throws(
-    () => parser("bar"), // This is line 25
-    /^25 \| bar/m
+    () => parser("bar"), // This is line 26
+    /^26 \| bar/m
   );
 });
 
@@ -41,21 +42,36 @@ test("with options", () => {
   assert(events.length > 0);
 });
 
-test("with library", async() => {
+test("with library", async t => {
+  if (!semver.satisfies(process.version, ">=20.8")) {
+    t.skip("Requires node 20.8+");
+    return;
+  }
+
   const parser = await withImports`
     import Foo from "./fixtures/foo.js"
     Bar = Foo`;
 
   assert.equal(parser("foo"), "foo");
 });
 
-test("with library, errors", async() => {
+test("with library, errors", async t => {
+  if (!semver.satisfies(process.version, ">=20.8")) {
+    t.skip("Requires node 20.8+");
+    return;
+  }
+
   await assert.rejects(() => withImports`
     import Boo from "./fixtures/foo.js"
     Bar = Foo`);
 });
 
-test("with library and options", async() => {
+test("with library and options", async t => {
+  if (!semver.satisfies(process.version, ">=20.8")) {
+    t.skip("Requires node 20.8+");
+    return;
+  }
+
   const peg = peggy.withImportsOptions({ trace: true });
   const parser = await peg`
     import Foo from "./fixtures/foo.js"