Skip to content

Yarn install is including tests and docs folder #3261

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

Closed
Sunyang730 opened this issue Apr 26, 2017 · 3 comments
Closed

Yarn install is including tests and docs folder #3261

Sunyang730 opened this issue Apr 26, 2017 · 3 comments

Comments

@Sunyang730
Copy link

Do you want to request a feature or report a bug?

feature

What is the current behavior?
I only tried this on a private repo, where yarn is including tests and docs folder when yarn install

If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?
To not include tests and docs folder

Please mention your node.js, yarn and operating system version.
node.js v7.9.0, OSX 10.11.6

@rally25rs
Copy link
Contributor

I'm pretty sure there is a duplicate bug around here somewhere but I'm having a hard time finding it...

IIRC, Yarn only filters the files listed in package.json when it publishes a package. On extraction, or copy from a local repo, it does not clean up files that are not in the files listing.

NPM on the other hand, seems to filter the files both on publish and when copying the files into node_modules during install.

@rally25rs
Copy link
Contributor

Ah, found it: duplicate of #2822

@voxsim
Copy link
Contributor

voxsim commented May 2, 2017

I close this issue because it is a duplicate, thanks @rally25rs ;)
@Sunyang730 if you have additional comments, please report it in the original issue!

@voxsim voxsim closed this as completed May 2, 2017
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

No branches or pull requests

4 participants