Skip to content

v50.7.0 may break in node 18; should note this in release notes #1393

Closed
@marcustyphoon

Description

@marcustyphoon

Due to—presumably?—the are-docs-informative major version bump in #1391, I get this error when running eslint 8 in Node 18:

ESLint: 8.57.1

Error [ERR_REQUIRE_ESM]: require() of ES Module /Users/marcus/Documents/GitHub/discretize/discretize-gear-optimizer-vite/node_modules/.pnpm/[email protected]/node_modules/are-docs-informative/lib/index.js from /Users/marcus/Documents/GitHub/discretize/discretize-gear-optimizer-vite/node_modules/.pnpm/[email protected][email protected]/node_modules/eslint-plugin-jsdoc/dist/rules/informativeDocs.cjs not supported.
Instead change the require of index.js in /Users/marcus/Documents/GitHub/discretize/discretize-gear-optimizer-vite/node_modules/.pnpm/[email protected][email protected]/node_modules/eslint-plugin-jsdoc/dist/rules/informativeDocs.cjs to a dynamic import() which is available in all CommonJS modules.

Updating node to 20+ fixes this, but this isn't documented in the release notes.

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions