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

Explicitly define contents of npm package #161

Merged
merged 1 commit into from
Jul 30, 2013
Merged

Explicitly define contents of npm package #161

merged 1 commit into from
Jul 30, 2013

Conversation

sindresorhus
Copy link
Contributor

No description provided.

@tj
Copy link
Owner

tj commented Jul 24, 2013

is this new?

@sindresorhus
Copy link
Contributor Author

No, it's been in npm for at least a couple of years: isaacs/npm@144b26a#L9R0

I recently discovered it and now much prefer to explicitly define what goes into my packages than having a long list of ignores in a separate file. Hopefully you'll agree.

@jednano
Copy link

jednano commented Jul 30, 2013

I'm going to have to start doing this now. Thanks @sindresorhus.

@tj
Copy link
Owner

tj commented Jul 30, 2013

weird!

tj added a commit that referenced this pull request Jul 30, 2013
Explicitly define contents of npm package
@tj tj merged commit 38b7adb into tj:master Jul 30, 2013
@sindresorhus sindresorhus deleted the files-prop branch July 31, 2013 08:15
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.

3 participants