search

fusionjs / fusion-cli

Migrated to https://github.com/fusionjs/fusionjs
MIT License
140 stars 37 forks source link

Add file list to package.json #739

Open madhusudhand opened 5 years ago

madhusudhand commented 5 years ago

fusion-cli npm modules includes all the files from source which makes it heavy. Hence included only required files.

CLAassistant commented 5 years ago

CLA assistant check
All committers have signed the CLA.

madhusudhand commented 5 years ago

@KevinGrandon added a generic negate rule "!**/test/" to ignore all the test directories under any directory not just 'build'. In case if we add test files for others later this will take care.

madhusudhand commented 5 years ago

Removed the optional entries README.md and LICENSE as per the review comments.

rtsao commented 5 years ago

I think sw.js is also missing. This leaves me thinking maybe we might want to use .npmignore instead, which would be the contents of .gitignore plus:

docs/
test/e2e
.github/
.buildkite/
.*

I'm wary of accidentally having necessary files being published, so perhaps it's better to instead explicitly say which files should not be published.