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

"core" folder missing on NPM registry (not a Lerna issue) #1932

Closed
TheOptimisticFactory opened this issue Feb 13, 2019 · 5 comments
Closed

Comments

@TheOptimisticFactory
Copy link

TheOptimisticFactory commented Feb 13, 2019

DISCLAIMER: this is not a bug from Lerna but an issue arising from one of its dependency => npm-packlist

Expected Behavior

  • A few days ago, publishing a project containing any folder named core would lead to all non-blacklisted files to be included in the tarball sent onto NPM registry

Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

  1. Have a project containing a folder named core anywhere in the tree with a file "path/to/core/index.js"
  2. Using Lerna 3.0, path/to/core and any of its content will be included in the tarball on publish
  3. Using the latest Lerna revision, path/to/core and any of its content will be missing from the tarball on publish
@evocateur
Copy link
Member

Yikes, that's a nasty surprise. Thanks for the heads up and opening the issue upstream!

@evocateur
Copy link
Member

(I'll leave this open until we get some resolution on npm/npm-packlist#24)

@TheOptimisticFactory
Copy link
Author

The upstream issue has been resolved with: npm/npm-packlist@9230bb8

-- It was also impacting NPM v6.8.0

This fix got released in npm/npm-packlist v1.4.0 and onward 🎉

DanElliottPalmer added a commit to cnduk/merlin-www-components that referenced this issue Feb 22, 2019
this is to workaround an issue where core folders were not being published. this
is due to lerna/lerna#1932
@Jeach
Copy link

Jeach commented Mar 9, 2019

Had the same issue with NPM v6.8.0 ... but fortunately, I knew I had a version I could update to, so when I updated (ie: 'sudo npm install -g npm'), it installed v6.9.0 and the problem seems to have been resolved. It now includes my 'core' sub-directory.

@evocateur
Copy link
Member

Update to lerna v3.13.1 and you will no longer encounter this problem when running lerna publish

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

3 participants