search

apostrophecms / random-words

Generate one or more common English words. Intended for use as sample text, for example generating random blog posts for testing
MIT License
245 stars 70 forks source link

add mocha as devDependency to lower package size #18

Closed thechickennagget closed 4 years ago

lwillmeth commented 4 years ago

Moving mocha to devDependencies is a good change, but why install mocha in npm test?

boutell commented 4 years ago

Yeah, that is superfluous. Just move it to devDependencies.

thechickennagget commented 4 years ago

i like this package but i hate that it uses "mocha" that has another 24 Dependencies https://www.npmjs.com/package/mocha?activeTab=dependencies

thechickennagget commented 4 years ago

Moving mocha to devDependencies is a good change, but why install mocha in npm test?

Alright i removed the npm install in test script