Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace is-core-module with isBuiltin from node:module #76

Open
ericallam opened this issue Aug 9, 2024 · 2 comments
Open

Replace is-core-module with isBuiltin from node:module #76

ericallam opened this issue Aug 9, 2024 · 2 comments
Labels
umbrella issue This issue contains a list of dependents of a package and tracks the progress in each

Comments

@ericallam
Copy link

No description provided.

@Fuzzyma Fuzzyma added the umbrella issue This issue contains a list of dependents of a package and tracks the progress in each label Dec 18, 2024
@pralkarz
Copy link

npx github:Fuzzyma/e18e-tools is-core-module -n 10 -q -o md -U https://npm.devminer.xyz/registry

# Downloads Traffic Version Package
1 356.09M 3588.98 GB ^2.13.0 resolve
2 100.05M 1008.41 GB ^2.13.0 eslint-import-resolver-node
3 97.75M 985.22 GB ^2.15.1 eslint-plugin-import
4 4.20M 42.32 GB ^2.12.0 depcheck
5 4.13M 41.67 GB ^2.13.1 eslint-import-resolver-webpack
6 732.43k 7.38 GB 2.9.0 pkg
7 19.45k 196.03 MB 2.9.0 @useparagon/pkg
8 11.89k 119.84 MB ^2.13.1 eslint-plugin-file-extension-in-import-ts
9 10.49k 105.77 MB ^2.15.1 vite-plugin-mock-dev-server
10 9.47k 95.49 MB ^2.2.0 dts-packer
11 4.90k 49.41 MB ^2.5.0 eslint-import-resolver-enhanced-resolve

isBultin requires Node 16.17.0+, therefore:

  • no. 1, 2, 3, 5 can't be migrated because they support Node >= 0.4
  • no. 4 supports Node >= 10, so it's also a no-go
  • no. 6 has been archived and deprecated a year ago.

The rest are very low-impact. Do we want to focus on them nonetheless or close this issue?

@43081j
Copy link
Collaborator

43081j commented Jan 26, 2025

we can also move some consumers of those lint plugins to alternatives like eslint-plugin-i

that'll bring a whole load of savings with it beyond is-core-module

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
umbrella issue This issue contains a list of dependents of a package and tracks the progress in each
Projects
None yet
Development

No branches or pull requests

4 participants