simplify .npmignore

[EdJoPaTo]

When looking at the current package content of 4.0.1 there are some not relevant files included. For example there is a 270 kB yarn-error.log or the .vscode folder:

$ ls -a1
.vscode
CONTRIBUTING.md
dist
lib
LICENSE
package.json
README.md
release-notes.md
rollup.config.js
runtime.js
yarn-error.log

This could be solved by adding these to the .npmignore. But the package.json files entry seems to be a much better solution here.

Using npm pack its possible to view the package content:

$ npm pack
npm notice 
npm notice 📦  diff@4.0.1
npm notice === Tarball Contents === 
npm notice 1.6kB  LICENSE                      
npm notice 20.2kB lib/patch/apply.js           
npm notice 2.3kB  lib/diff/array.js            
npm notice 2.0kB  lib/util/array.js            
npm notice 30.9kB lib/diff/base.js             
npm notice 1.7kB  lib/diff/character.js        
npm notice 21.3kB lib/patch/create.js          
npm notice 2.0kB  lib/diff/css.js              
npm notice 46.8kB dist/diff.js                 
npm notice 4.6kB  lib/util/distance-iterator.js
npm notice 2.3kB  lib/convert/dmp.js           
npm notice 43.5kB lib/index.es6.js             
npm notice 7.2kB  lib/index.js                 
npm notice 12.9kB lib/diff/json.js             
npm notice 5.7kB  lib/diff/line.js             
npm notice 50.4kB lib/patch/merge.js           
npm notice 1.8kB  lib/util/params.js           
npm notice 16.6kB lib/patch/parse.js           
npm notice 2.1kB  lib/diff/sentence.js         
npm notice 8.5kB  lib/diff/word.js             
npm notice 3.4kB  lib/convert/xml.js           
npm notice 2.1kB  package.json                 
npm notice 8.8kB  README.md                    
npm notice === Tarball Details === 
npm notice name:          diff                                    
npm notice version:       4.0.1                                   
npm notice filename:      diff-4.0.1.tgz                          
npm notice package size:  89.4 kB                                 
npm notice unpacked size: 298.5 kB                                
npm notice shasum:        fc009ded112af9000da6ab6779f0e9fc98cd70bb
npm notice integrity:     sha512-X42lopbs/sTd7[...]MagdIhyd5v+8Q==
npm notice total files:   23                                      
npm notice 
diff-4.0.1.tgz

I am not sure if the .js files in the root folder are needed in production, currently I assume they are not needed in the package.

The release-notes.md will be missing out too. I can add this to the 'files' entry if wished.
CONTRIBUTING.md does not seem appropriate in the packaged files.

Unrelated: with using the CONTRIBUTING.md instructions I do not get the dist/diff.min.js file. This file is missing in my npm pack run too.

[EdJoPaTo]

When digging in the history I found #19 adding this feature ages ago and 287994c which removed it again

[ffflorian]

@kpdecker any update on this?

[kpdecker]

Sorry for delay on this. Everytime I look, I become concerned about the scope of these changes and don't have time to work through if this will bite in future.

Give 287994c, not remembering the details of a 5 year old commit, and that this project is in maintenance mode, I don't want to take on the risk of switching this. Glad to take a PR that removes any files from the package via the npmignore though.