fix: compatibility with Node 12 (#253)

Author: ota-meshi

.github/workflows/ci.yml

@@ -106,7 +106,13 @@ jobs:
               run: npm run build
 
             - name: ▶️ Run test script
-              run: npm run test
+              run: |
+                  npm run test
+
+                  # Load src/index.js to check for syntax errors.
+                  # This is because for some reason the exit code does not become 1 in Node.js 12.
+                  # See https://github.com/eslint-community/eslint-utils/pull/253
+                  node src/index.mjs
 
             - name: ⬆️ Upload coverage report
               uses: codecov/codecov-action@v4

src/get-function-head-location.mjs

@@ -51,10 +51,10 @@ export function getFunctionHeadLocation(node, sourceCode) {
         parent.type === "PropertyDefinition"
     ) {
         start = /** @type {SourceLocation} */ (parent.loc).start
-        end = getOpeningParenOfParams(node, sourceCode)?.loc.start
+        end = getOpeningParenOfParams(node, sourceCode).loc.start
     } else {
         start = /** @type {SourceLocation} */ (node.loc).start
-        end = getOpeningParenOfParams(node, sourceCode)?.loc.start
+        end = getOpeningParenOfParams(node, sourceCode).loc.start
     }
 
     return {