chore: a few review suggestions

Author: aladdin-add

.github/workflows/CI.yml

@@ -31,15 +31,12 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest, windows-latest, macOS-latest]
-        eslint: [8.x]
+        eslint: [8.x, "9.0.0-alpha.0"]
         node: [18.x, 20.x]
         include:
           - os: ubuntu-latest
             node: "21.x"
             eslint: "8.x"
-          - os: ubuntu-latest
-            node: "20.x"
-            eslint: "9.0.0-alpha.0"
     runs-on: ${{ matrix.os }}
     steps:
       - name: Checkout
@@ -51,7 +48,9 @@ jobs:
         with:
           node-version: ${{ matrix.node }}
       - name: Install Packages
-        run: npm install
+        run:
+            npm i npm@latest -g # use latest npm
+            npm install
       - name: Install ESLint ${{ matrix.eslint }}
         run: |
           npm install --no-save --force eslint@${{ matrix.eslint }}

tests/eslint-rule-tester.js

@@ -11,9 +11,7 @@ const globals = require("globals")
 // greater than or equal to ESLint v9
 exports.gteEslintV9 = +eslintVersion.split(".")[0] >= 9
 
-exports.FlatRuleTester = exports.gteEslintV9
-    ? RuleTester
-    : unofficialApis.FlatRuleTester
+exports.FlatRuleTester = exports.gteEslintV9 ? RuleTester : FlatRuleTester
 
 // to support the `env:{ es6: true, node: true}` rule-tester (env has been away in flat config.)
 // * enabled by default as it's most commonly used in the package.

tests/lib/rules/no-unpublished-require.js

@@ -5,7 +5,6 @@
 "use strict"
 
 const path = require("path")
-const globals = require("globals")
 const RuleTester = require("#eslint-rule-tester").RuleTester
 const rule = require("../../../lib/rules/no-unpublished-require")
 
@@ -182,7 +181,6 @@ ruleTester.run("no-unpublished-require", rule, {
         {
             filename: fixture("1/test.js"),
             code: "require(`foo${bar}`);",
-            languageOptions: { globals: globals.node },
         },
 
         // Should work fine if the filename is relative.