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

Remove all non-prod files from distribution #1364

Closed
wants to merge 1 commit into from
Closed

Remove all non-prod files from distribution #1364

wants to merge 1 commit into from

Conversation

jeffalder
Copy link

When installing ssh2, the tests and test fixtures are all installed. This doesn't feel desirable, as it puts test keys, example code, and other data into the runtime environment.

This change will pack only the lib directory, but that does include LICENSE, package.json, and README.md per npm. See https://docs.npmjs.com/cli/v10/using-npm/developers#keeping-files-out-of-your-package for more information.

Running npm pack gave me a tarball with these files:

package/LICENSE
package/lib/protocol/crypto/src/binding.cc
package/lib/protocol/crypto/binding.gyp
package/lib/agent.js
package/lib/Channel.js
package/lib/client.js
package/lib/protocol/constants.js
package/lib/protocol/crypto.js
package/lib/protocol/handlers.js
package/lib/protocol/handlers.misc.js
package/lib/http-agents.js
package/lib/index.js
package/lib/protocol/kex.js
package/lib/keygen.js
package/lib/protocol/keyParser.js
package/lib/protocol/node-fs-compat.js
package/lib/protocol/crypto/poly1305.js
package/lib/protocol/Protocol.js
package/lib/server.js
package/lib/protocol/SFTP.js
package/lib/protocol/utils.js
package/lib/utils.js
package/lib/protocol/zlib.js
package/package.json
package/README.md

Let me know if anything is missing.

@mscdex
Copy link
Owner

mscdex commented Feb 8, 2024

Duplicate of #1217

@mscdex mscdex marked this as a duplicate of #1217 Feb 8, 2024
@mscdex mscdex closed this Feb 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants