Skip to content
New issue

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

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

Already on GitHub? # to your account

limit the number of files to be included in the build #1542

Closed
wants to merge 1 commit into from

Conversation

mef
Copy link
Contributor

@mef mef commented Mar 9, 2024

This is a proposal to fix #1541.

Changes:

  • Move all dependencies to devDependencies instead of dependencies, so that they are not included when running npm install intl-tel-input.
  • Only include the build and the documentation inside the npm package.

This dramatically decreases the package size, making it faster to install, and consuming 100x less disk space (1.2MB instead of 122 MB).

@jackocnr
Copy link
Owner

Thanks very much for this. I've merged the commit manually and added a further commit with some fixes. I'm just waiting for Travis to fix my testing account to check all ok there, and then I will do a new release including this. Thanks again.

@jackocnr jackocnr closed this Mar 14, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Decrease npm package size
2 participants