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

Publish non-private subdirectories when root is private #483

Merged
merged 2 commits into from Feb 2, 2020
Merged

Publish non-private subdirectories when root is private #483

merged 2 commits into from Feb 2, 2020

Conversation

lukeggchapman
Copy link
Contributor

Fixes #474

I wanted to write tests, but there doesn't seem to be an example of mocking out package.json contents anywhere. If you think it's necessary, I'll revisit when I can, and invest time in setting that up.

@itaisteinherz itaisteinherz changed the title Non-private options.contents are now published Publish non-private subdirectories when root is private Jan 11, 2020
source/util.js Outdated Show resolved Hide resolved
Copy link
Collaborator

@itaisteinherz itaisteinherz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, though I'd like to wait for @sindresorhus's review before merging this.

@sindresorhus sindresorhus merged commit 607e5c1 into sindresorhus:master Feb 2, 2020
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

Successfully merging this pull request may close these issues.

Private root package prevents non-private subdirectory from being published
3 participants