chore(deps-dev): bump eslint-plugin-n from 17.24.0 to 18.2.1#145
chore(deps-dev): bump eslint-plugin-n from 17.24.0 to 18.2.1#145dependabot[bot] wants to merge 1 commit into
Conversation
LabelsThe following labels could not be found: Please fix the above issues or remove invalid values from |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7e3ba77a9d
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| "eslint": "^10.0.2", | ||
| "eslint-plugin-import-x": "^4.12.2", | ||
| "eslint-plugin-n": "^17.24.0", | ||
| "eslint-plugin-n": "^18.1.0", |
There was a problem hiding this comment.
Handle the ESM-only plugin export before bumping
With this dependency line, yarn lint loads eslint.config.js, which still does const eslintPluginN = require('eslint-plugin-n') and then reads eslintPluginN.configs. In v18 the plugin entrypoint is ESM and default-exports the plugin object, so on modern Node require() returns a namespace object whose configs are under .default (and older Node versions throw ERR_REQUIRE_ESM); the Build workflow's lint job will fail while evaluating the config before linting anything unless the config is converted/updated or this bump is held back.
Useful? React with 👍 / 👎.
Overall package sizeSelf size: 30.06 MB Dependency sizes| name | version | self size | total size | |------|---------|-----------|------------|🤖 This report was automatically generated by heaviest-objects-in-the-universe |
This comment has been minimized.
This comment has been minimized.
7e3ba77 to
75cbb2f
Compare
Bumps [eslint-plugin-n](https://github.com/eslint-community/eslint-plugin-n) from 17.24.0 to 18.2.1. - [Release notes](https://github.com/eslint-community/eslint-plugin-n/releases) - [Changelog](https://github.com/eslint-community/eslint-plugin-n/blob/master/CHANGELOG.md) - [Commits](eslint-community/eslint-plugin-n@v17.24.0...v18.2.1) --- updated-dependencies: - dependency-name: eslint-plugin-n dependency-version: 18.1.0 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
75cbb2f to
fc06c57
Compare
Bumps eslint-plugin-n from 17.24.0 to 18.2.1.
Release notes
Sourced from eslint-plugin-n's releases.
... (truncated)
Changelog
Sourced from eslint-plugin-n's changelog.
... (truncated)
Commits
37da568chore(master): release 18.2.1 (#546)a0d7819fix: ensure types have valid syntax (#544)ccd1832chore(master): release 18.2.0 (#543)04c99dffeat: addmodule.exportsforrequire(esm)interop (#542)fc70bdfchore(master): release 18.1.0 (#528)fd4f84efeat: Allow workspace root dependencies for no-extraneous-import (#536)5fde036fix: Allow leading forwardslash in package.json "files" field (#534) (#535)63a90effix: no-extraneous-import for type-only@typesdependencies (#533)75fbe34fix(prefer-promises/fs): add missing fs.promises APIs (cp, glob, lutimes, ope...058916adocs: Update online playground link in README