Skip to content

Removes a few files from the installed package #1738

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pieterocp
Copy link

Since as far as I am aware, these are helpful when developing, but not too helpful when installed.

(and removed a few off the guidance from https://docs.npmjs.com/cli/v11/using-npm/developers#keeping-files-out-of-your-package)

Tested with a npm pack && tar -xvzf *.tgz && rm -rf package *.tgz which should detail what files are left over (thanks to Jeff D /@jdx .

Since as far as I am aware, these are helpful when developing, but not too helpful when in use

(and removed a few off the guidance from https://docs.npmjs.com/cli/v11/using-npm/developers#keeping-files-out-of-your-package)
@pieterocp
Copy link
Author

pieterocp commented May 9, 2025

Dear contributors, can someone review this? It's pretty small update but does add up to send less traffic whenever someone does install this package.

(I spotted these when having a look at some CLI output packaging theses files. I hope this falls under Contributing Anything else constructively helpful.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant