-
-
Notifications
You must be signed in to change notification settings - Fork 2
Replace is-ci
with ci-info
.isCI
#128
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
Comments
Replaced it in Cypress: cypress-io/cypress#30465 |
Replaced it in |
Here is a list of the top 20 packages using is-ci according to npm.js (which does not contain all dependents because it only allows the first 396 to be crawled): Stats are downloads last-month.
|
Could be a good couple of PRs! |
PR to electron-builder: electron-userland/electron-builder#9117 |
FYI, lerna is still using it as well dependants as of 12.05.2025 (I updated the metadata mirror again).
|
Looks like ci-info ships a 6KB, 350 line unmimified JSON file in the npm package. Mimifying that file would shave a few bytes off the install size, which is probably worth it with it having 52 million downloads per week. Unfortunately it doesn't look like they have a proper build step there yet. The existing "build" script sorts the JSON file and creates the index.d.ts file, but they check the results into the git repo, which you wouldn't want to do for the mimified JSON file. |
is-ci
is literally just an export wrapper forci-info
'sisCI
field, so we can just replace it everywhere we see it. That'd save 3KB and 2 HTTP roundtrips during installation.The text was updated successfully, but these errors were encountered: