Configuring .npmignore #65

shama · 2013-09-15T06:01:59Z

I think we should configure the .npmignore files within the contrib plugins to only include relevant files to run the task. No docs, tests or Gruntfile.js. That way we can alleviate a little bit of the time and space it takes to install the tasks.

Any objections or something I'm missing by doing this?

/cc @cowboy @vladikoff @sindresorhus @jsoverson

The text was updated successfully, but these errors were encountered:

jsoverson · 2013-09-15T06:21:31Z

That's probably a good idea. Personally, I like having all that extra in npm modules so that I can debug more easily when necessary, but I doubt that's a common use.

sindresorhus · 2013-09-15T11:36:08Z

Definitely, but we should rather use the files property in package.json to include files, instead of ignoring.

https://npmjs.org/doc/json.html#files

Huge 👍 from me though.

shama · 2013-10-19T22:05:39Z

Ok, let's do that ^. @CaryLandholt let me know if you have any interest in doing this otherwise I'll write a script and take care of this soon. Thanks!

shama · 2013-10-19T22:10:45Z

Just for clarification, let's include:

{
  "files": [
    "tasks/",
    "AUTHORS",
    "CHANGELOG",
    "CONTRIBUTING.md",
    "LICENSE-MIT",
    "package.json",
    "README.md"
  ]
}

sindresorhus · 2013-10-19T22:12:39Z

no need for AUTHORS, CHANGELOG, CONTRIBUTING.md in the package. And package.json and readme.md is always included, so no need to specify.

shama · 2013-10-19T22:14:58Z

Cool, did not know that... so that makes it really simple:

{
  "files": [
    "tasks",
    "LICENSE-MIT"
  ]
}

EDIT: Removed the /. Thanks Sindre!

sindresorhus · 2013-10-19T22:30:20Z

yup, you can drop the / too.

shama · 2013-12-15T18:34:15Z

The deed is done.

sindresorhus · 2013-12-15T18:36:03Z

Hah, nice. Thanks @shama

cowboy · 2013-12-16T16:27:24Z

Yay!

…merge-from-upstream * 'master' of github.com:gruntjs/grunt-contrib-qunit: (29 commits) Update copyright to 2014 Style and Doc Fixes Squashed commit of the following: 0.4.0 Update grunt-lib-phantomjs to v0.5.0. Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65 Update qunit-overview.md, include CentOS dependencies. Closes asciidiscogh-49 0.3.0 Fixes asciidiscoGH-47. Update to latest phantom dependency. Remove Gruntfile.js as main in package.json removing confusing error message Propagate error.onError events from Phantom. Closes asciidiscoGH-11. Update README Update readme to document PhantomJS OS dependencies. Fixes asciidisco#43 Revert "Cleanup trailing whitespace in docs". Necessary for hard linebreaks. Cleanup trailing whitespace in docs Add `force` option Update QUnit to latest; cleanup QUnit markup Add new events for fail conditions, so that reporters can still report on the test Bump v0.2.2 ... Conflicts: CHANGELOG Gruntfile.js README.md package.json phantomjs/bridge.js tasks/qunit.js

…:gruntjs/grunt-contrib-qunit into merge-upstream * github.com:gruntjs/grunt-contrib-qunit: (26 commits) 0.4.0 Update grunt-lib-phantomjs to v0.5.0. Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65 Update qunit-overview.md, include CentOS dependencies. Closes asciidiscogh-49 0.3.0 Fixes asciidiscoGH-47. Update to latest phantom dependency. Remove Gruntfile.js as main in package.json removing confusing error message Propagate error.onError events from Phantom. Closes asciidiscoGH-11. Update README Update readme to document PhantomJS OS dependencies. Fixes asciidisco#43 Revert "Cleanup trailing whitespace in docs". Necessary for hard linebreaks. Cleanup trailing whitespace in docs Add `force` option Update QUnit to latest; cleanup QUnit markup Add new events for fail conditions, so that reporters can still report on the test Bump v0.2.2 Remove broken link in docs Update devDeps Update copyright ... Conflicts: README.md package.json phantomjs/bridge.js tasks/qunit.js

shama mentioned this issue Sep 15, 2013

add .npmignore gruntjs/grunt-contrib-clean#35

Closed

shama mentioned this issue Oct 26, 2013

update ignore with Gruntfile gruntjs/grunt-contrib-sass#76

Closed

shama mentioned this issue Nov 30, 2013

Create .npmignore gruntjs/grunt-contrib-jshint#124

Closed

shama added a commit to gruntjs/grunt-contrib-stylus that referenced this issue Dec 3, 2013

Set files to publish to npm. Ref gruntjs/gruntjs.com#65

191c374

shama added a commit to gruntjs/grunt-contrib-bump that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

32e7bd4

shama added a commit to gruntjs/grunt-contrib-clean that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

cc125bd

shama added a commit to gruntjs/grunt-contrib-coffee that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

581649c

shama added a commit to gruntjs/grunt-contrib-compass that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

0874efb

shama added a commit to gruntjs/grunt-contrib-compress that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

abf2482

shama added a commit to gruntjs/grunt-contrib-concat that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

9ba6287

shama added a commit to gruntjs/grunt-contrib-connect that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

8345d35

shama added a commit to gruntjs/grunt-contrib-copy that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

c3441d9

shama added a commit to gruntjs/grunt-contrib-cssmin that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

77dbbc2

shama added a commit to gruntjs/grunt-contrib-handlebars that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

c04275a

shama added a commit to gruntjs/grunt-contrib-imagemin that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

4f3e8e3

shama added a commit to gruntjs/grunt-contrib-jasmine that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

965da97

shama added a commit to gruntjs/grunt-contrib-pug that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

1c09199

shama added a commit to gruntjs/grunt-contrib-jst that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

a5c76f5

shama added a commit to gruntjs/grunt-contrib-less that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

710c462

shama added a commit to gruntjs/grunt-contrib-nodeunit that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

be0ceed

shama added a commit to gruntjs/grunt-contrib-qunit that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

f9f5e87

shama added a commit to gruntjs/grunt-contrib-requirejs that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

601ee2d

shama added a commit to gruntjs/grunt-contrib-stylus that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

f5d52d0

shama added a commit to gruntjs/grunt-contrib-symlink that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

b08109f

shama added a commit to gruntjs/grunt-contrib-uglify that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

806cb94

shama added a commit to gruntjs/grunt-contrib-yuidoc that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

9b3c8ed

shama added a commit to gruntjs/grunt-contrib-watch that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

b884948

shama closed this as completed Dec 15, 2013

shama added a commit to gruntjs/grunt-contrib-csslint that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

9622739

shama added a commit to gruntjs/grunt-contrib-htmlmin that referenced this issue Dec 15, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

262badf

paazmaya added a commit to paazmaya/grunt-image-profile that referenced this issue Dec 16, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

f5df1d4

paazmaya added a commit to paazmaya/grunt-hgactivity that referenced this issue Dec 16, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

e1e8dff

paazmaya added a commit to paazmaya/grunt-html2md that referenced this issue Dec 16, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

66147a6

johnnyfreeman pushed a commit to johnnyfreeman/grunt-contrib-coffee that referenced this issue Dec 20, 2013

Explicitly set files to publish to npm. Ref gruntjs/gruntjs.com#65

59a2e09

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Configuring .npmignore #65

Configuring .npmignore #65

shama commented Sep 15, 2013

jsoverson commented Sep 15, 2013

sindresorhus commented Sep 15, 2013

shama commented Oct 19, 2013

shama commented Oct 19, 2013

sindresorhus commented Oct 19, 2013

shama commented Oct 19, 2013

sindresorhus commented Oct 19, 2013

shama commented Dec 15, 2013

sindresorhus commented Dec 15, 2013

cowboy commented Dec 16, 2013

Configuring .npmignore #65

Configuring .npmignore #65

Comments

shama commented Sep 15, 2013

jsoverson commented Sep 15, 2013

sindresorhus commented Sep 15, 2013

shama commented Oct 19, 2013

shama commented Oct 19, 2013

sindresorhus commented Oct 19, 2013

shama commented Oct 19, 2013

sindresorhus commented Oct 19, 2013

shama commented Dec 15, 2013

sindresorhus commented Dec 15, 2013

cowboy commented Dec 16, 2013