[
https://issues.apache.org/jira/browse/THRIFT-3485?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15055120#comment-15055120
]
ASF GitHub Bot commented on THRIFT-3485:
----------------------------------------
Github user azylman commented on the pull request:
https://github.com/apache/thrift/pull/741#issuecomment-164294334
@nsuke Looks like that's not published in 0.9.3. I just confirmed that
master does _not_ have this problem, making this PR unnecessary. Hopefully that
change can get published soon :/
> thrift NPM package includes whole thrift repository, is 30 megabytes
> --------------------------------------------------------------------
>
> Key: THRIFT-3485
> URL: https://issues.apache.org/jira/browse/THRIFT-3485
> Project: Thrift
> Issue Type: Bug
> Components: Node.js - Library
> Affects Versions: 0.9.3
> Reporter: Alex Zylman
> Original Estimate: 0.25h
> Remaining Estimate: 0.25h
>
> The thrift npm package is as big as all of our other node dependencies
> combined. It currently publishes the entire thrift repository, which is ~30mb
> right now. As far as I can tell, it only requires lib/nodejs, which is 376k.
> I propose adding an .npmignore file. You can more info in the npm
> documentation:
> https://docs.npmjs.com/misc/developers#keeping-files-out-of-your-package
> If the package only requires lib/nodejs, we should be able to add an entry to
> the npmignore file for every folder (including lib), and then a negated entry
> for lib/nodejs. We'll also want to include everything from gitignore, because
> npm will no longer use that once an npmignore file is added.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
