boybundit / linebot

🤖 SDK for the LINE Messaging API for Node.js
https://www.npmjs.com/package/linebot
MIT License
217 stars 93 forks source link

Update package.json file to exclude unnecessary files #4

Closed boybundit closed 8 years ago

boybundit commented 8 years ago

Like test and CI config file

boybundit commented 8 years ago

Let's use package.json instead of .npmignore

xRahul commented 8 years ago

Hey @boybundit

You would have to use .npmignore to exclude files, as the files property in package.json is used to include them, not exclude. We can add all the folders and files we want to include in package.json, but that would have to be updated with every new file created, which would be unnecessary work. Please check

boybundit commented 8 years ago

@xRahul Thanks for the suggestion. I was thinking the same, but decided to use package.json just because it does not require yet another config file.

I might change my mind later if it breaks because we forget to update package.json....

xRahul commented 8 years ago

@boybundit alright!