Improve and update `packages.json`

vitkarpov / grunt-nunjucks-2-html

Compiles nunjucks templates *right* into HTML

MIT License

35 stars 11 forks source link

Improve and update `packages.json` #30

Closed ArmorDarks closed 9 years ago

ArmorDarks commented 9 years ago

Moved peerDependency into devDependency, since peerDependency depreciated
Moved lodash to dependencides — it's needed for production, not only for development
Proper format for license
Updated dependencies' versions
A bit more meaningful order of properties

ArmorDarks commented 9 years ago

I propose to drop license's filename, otherwise we have to use literally that value:

"license": "SEE LICENSE IN <LICENSE>",

In all other cases npm will report error during parsing license, since he strongly ensured that MIT is standard license and there is no need to point to local file manually: https://docs.npmjs.com/files/package.json#license

vitkarpov commented 9 years ago

Drop license — +1

vitkarpov commented 9 years ago

Please, remove lodash from dependencies. I'm about to remove cloneDeep 'cause of performance issues you've pointed.

vitkarpov commented 9 years ago

let's fix the latter and :+1:

ArmorDarks commented 9 years ago

done, but there will be conflict with 49fcaf76fed59d0dfb829d2b59e70372ec2b5f72 anyway which must be resolved manually

vitkarpov commented 9 years ago

done