From 9ccdcb758f37ae5efe464699a5442d98cf1f73f3 Mon Sep 17 00:00:00 2001 From: Greg Walker Date: Thu, 6 May 2021 14:55:00 -0500 Subject: [PATCH] [Refactor] switch to an internal replacement for `pkg-up` and `read-pkg-up` --- CHANGELOG.md | 9 +++++++-- package.json | 3 --- src/core/packagePath.js | 8 ++++---- src/rules/no-extraneous-dependencies.js | 4 ++-- src/rules/no-import-module-exports.js | 4 ++-- src/rules/no-relative-packages.js | 4 ++-- src/rules/no-unused-modules.js | 4 ++-- tests/files/package.json | 2 +- utils/readPkgUp.js | 2 +- 9 files changed, 21 insertions(+), 19 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 9203739fd..26a6e76fb 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -6,6 +6,9 @@ This change log adheres to standards from [Keep a CHANGELOG](http://keepachangel ## [Unreleased] +### Changed +- [Refactor] switch to an internal replacement for `pkg-up` and `read-pkg-up` ([#2047], [@mgwalker]) + ### Added - [`no-unresolved`]: add `caseSensitiveStrict` option ([#1262], thanks [@sergei-startsev]) - [`no-unused-modules`]: add eslint v8 support ([#2194], thanks [@coderaiser]) @@ -915,7 +918,7 @@ for info on changes for earlier releases. [#2160]: https://github.com/import-js/eslint-plugin-import/pull/2160 [#2158]: https://github.com/import-js/eslint-plugin-import/pull/2158 [#2156]: https://github.com/import-js/eslint-plugin-import/pull/2156 -[#2149]: https://github.com/benmosher/eslint-plugin-import/pull/2149 +[#2149]: https://github.com/import-js/eslint-plugin-import/pull/2149 [#2146]: https://github.com/import-js/eslint-plugin-import/pull/2146 [#2140]: https://github.com/import-js/eslint-plugin-import/pull/2140 [#2138]: https://github.com/import-js/eslint-plugin-import/pull/2138 @@ -928,6 +931,7 @@ for info on changes for earlier releases. [#2083]: https://github.com/import-js/eslint-plugin-import/pull/2083 [#2075]: https://github.com/import-js/eslint-plugin-import/pull/2075 [#2071]: https://github.com/import-js/eslint-plugin-import/pull/2071 +[#2047]: https://github.com/import-js/eslint-plugin-import/pull/2047 [#2034]: https://github.com/import-js/eslint-plugin-import/pull/2034 [#2028]: https://github.com/import-js/eslint-plugin-import/pull/2028 [#2026]: https://github.com/import-js/eslint-plugin-import/pull/2026 @@ -1172,8 +1176,8 @@ for info on changes for earlier releases. [#2161]: https://github.com/import-js/eslint-plugin-import/issues/2161 [#2118]: https://github.com/import-js/eslint-plugin-import/issues/2118 [#2067]: https://github.com/import-js/eslint-plugin-import/issues/2067 -[#2056]: https://github.com/import-js/eslint-plugin-import/issues/2056 [#2063]: https://github.com/import-js/eslint-plugin-import/issues/2063 +[#2056]: https://github.com/import-js/eslint-plugin-import/issues/2056 [#1998]: https://github.com/import-js/eslint-plugin-import/issues/1998 [#1965]: https://github.com/import-js/eslint-plugin-import/issues/1965 [#1924]: https://github.com/import-js/eslint-plugin-import/issues/1924 @@ -1492,6 +1496,7 @@ for info on changes for earlier releases. [@Maxim-Mazurok]: https://github.com/Maxim-Mazurok [@maxkomarychev]: https://github.com/maxkomarychev [@maxmalov]: https://github.com/maxmalov +[@mgwalker]: https://github.com/mgwalker [@MikeyBeLike]: https://github.com/MikeyBeLike [@mplewis]: https://github.com/mplewis [@nickofthyme]: https://github.com/nickofthyme diff --git a/package.json b/package.json index b845b609d..08d5699f6 100644 --- a/package.json +++ b/package.json @@ -104,14 +104,11 @@ "doctrine": "^2.1.0", "eslint-import-resolver-node": "^0.3.6", "eslint-module-utils": "^2.6.2", - "find-up": "^2.0.0", "has": "^1.0.3", "is-core-module": "^2.6.0", "is-glob": "^4.0.1", "minimatch": "^3.0.4", "object.values": "^1.1.4", - "pkg-up": "^2.0.0", - "read-pkg-up": "^3.0.0", "resolve": "^1.20.0", "tsconfig-paths": "^3.11.0" } diff --git a/src/core/packagePath.js b/src/core/packagePath.js index a8c3c6763..2b5a2d41e 100644 --- a/src/core/packagePath.js +++ b/src/core/packagePath.js @@ -1,6 +1,6 @@ import { dirname } from 'path'; -import findUp from 'find-up'; -import readPkgUp from 'read-pkg-up'; +import pkgUp from 'eslint-module-utils/pkgUp'; +import readPkgUp from 'eslint-module-utils/readPkgUp'; export function getContextPackagePath(context) { @@ -8,12 +8,12 @@ export function getContextPackagePath(context) { } export function getFilePackagePath(filePath) { - const fp = findUp.sync('package.json', { cwd: filePath }); + const fp = pkgUp({ cwd: filePath }); return dirname(fp); } export function getFilePackageName(filePath) { - const { pkg, path } = readPkgUp.sync({ cwd: filePath, normalize: false }); + const { pkg, path } = readPkgUp({ cwd: filePath, normalize: false }); if (pkg) { // recursion in case of intermediate esm package.json without name found return pkg.name || getFilePackageName(dirname(dirname(path))); diff --git a/src/rules/no-extraneous-dependencies.js b/src/rules/no-extraneous-dependencies.js index 8d2e294cc..06a724a09 100644 --- a/src/rules/no-extraneous-dependencies.js +++ b/src/rules/no-extraneous-dependencies.js @@ -1,6 +1,6 @@ import path from 'path'; import fs from 'fs'; -import readPkgUp from 'read-pkg-up'; +import readPkgUp from 'eslint-module-utils/readPkgUp'; import minimatch from 'minimatch'; import resolve from 'eslint-module-utils/resolve'; import moduleVisitor from 'eslint-module-utils/moduleVisitor'; @@ -69,7 +69,7 @@ function getDependencies(context, packageDir) { Object.assign( packageContent, extractDepFields( - readPkgUp.sync({ cwd: context.getPhysicalFilename ? context.getPhysicalFilename() : context.getFilename(), normalize: false }).pkg + readPkgUp({ cwd: context.getPhysicalFilename ? context.getPhysicalFilename() : context.getFilename(), normalize: false }).pkg ) ); } diff --git a/src/rules/no-import-module-exports.js b/src/rules/no-import-module-exports.js index 50ba212c8..45710929c 100644 --- a/src/rules/no-import-module-exports.js +++ b/src/rules/no-import-module-exports.js @@ -1,9 +1,9 @@ import minimatch from 'minimatch'; import path from 'path'; -import pkgUp from 'pkg-up'; +import pkgUp from 'eslint-module-utils/pkgUp'; function getEntryPoint(context) { - const pkgPath = pkgUp.sync(context.getPhysicalFilename ? context.getPhysicalFilename() : context.getFilename()); + const pkgPath = pkgUp({ cwd: context.getPhysicalFilename ? context.getPhysicalFilename() : context.getFilename() }); try { return require.resolve(path.dirname(pkgPath)); } catch (error) { diff --git a/src/rules/no-relative-packages.js b/src/rules/no-relative-packages.js index 90c1ecc70..714eb3f5e 100644 --- a/src/rules/no-relative-packages.js +++ b/src/rules/no-relative-packages.js @@ -1,5 +1,5 @@ import path from 'path'; -import readPkgUp from 'read-pkg-up'; +import readPkgUp from 'eslint-module-utils/readPkgUp'; import resolve from 'eslint-module-utils/resolve'; import moduleVisitor, { makeOptionsSchema } from 'eslint-module-utils/moduleVisitor'; @@ -7,7 +7,7 @@ import importType from '../core/importType'; import docsUrl from '../docsUrl'; function findNamedPackage(filePath) { - const found = readPkgUp.sync({ cwd: filePath, normalize: false }); + const found = readPkgUp({ cwd: filePath }); if (found.pkg && !found.pkg.name) { return findNamedPackage(path.join(found.path, '../..')); } diff --git a/src/rules/no-unused-modules.js b/src/rules/no-unused-modules.js index ab347bd8c..068eb911c 100644 --- a/src/rules/no-unused-modules.js +++ b/src/rules/no-unused-modules.js @@ -10,7 +10,7 @@ import resolve from 'eslint-module-utils/resolve'; import visit from 'eslint-module-utils/visit'; import docsUrl from '../docsUrl'; import { dirname, join } from 'path'; -import readPkgUp from 'read-pkg-up'; +import readPkgUp from 'eslint-module-utils/readPkgUp'; import values from 'object.values'; import includes from 'array-includes'; @@ -352,7 +352,7 @@ const newDefaultImportExists = specifiers => specifiers.some(({ type }) => type === IMPORT_DEFAULT_SPECIFIER); const fileIsInPkg = file => { - const { path, pkg } = readPkgUp.sync({ cwd: file, normalize: false }); + const { path, pkg } = readPkgUp({ cwd: file }); const basePath = dirname(path); const checkPkgFieldString = pkgField => { diff --git a/tests/files/package.json b/tests/files/package.json index de1d80275..365f02b6e 100644 --- a/tests/files/package.json +++ b/tests/files/package.json @@ -12,7 +12,7 @@ "esm-package": "^1.0.0", "jquery": "^3.1.0", "lodash.cond": "^4.3.0", - "pkg-up": "^1.0.0", + "find-up": "^1.0.0", "rxjs": "^1.0.0" }, "optionalDependencies": { diff --git a/utils/readPkgUp.js b/utils/readPkgUp.js index 245afde68..6a6a1eea3 100644 --- a/utils/readPkgUp.js +++ b/utils/readPkgUp.js @@ -35,7 +35,7 @@ function stripBOM(str) { * THE SOFTWARE. */ exports.default = function readPkgUp(opts) { - const fp = pkgUp(opts.cwd); + const fp = pkgUp(opts); if (!fp) { return {};