Modify lodash importing

[abetomo]

No description provided.

[dschep]

I was wondering why this worked, bc I depend on the modular versions of those functions in lodash. But it seems (unsurprisingly in hindsight) that lodash is pulled in by the dependency tree anyway.

So, could you also replace the lodash.get, lodash.set and lodash.values in package.json with lodash?

[kichik]

lodash seems to be pulled in because of eslint and archiver.

C:\Dev\serverless-python-requirements>yarn why lodash
yarn why v1.5.1
[1/4] Why do we have the module "lodash"...?
[2/4] Initialising dependency graph...
[3/4] Finding dependency...
[4/4] Calculating file sizes...
=> Found "lodash@4.17.5"
info Reasons this module exists
   - "eslint" depends on it
   - Hoisted from "eslint#lodash"
   - Hoisted from "archiver#lodash"
   - Hoisted from "eslint#table#lodash"
   - Hoisted from "archiver#async#lodash"
   - Hoisted from "eslint#inquirer#lodash"
   - Hoisted from "archiver#zip-stream#lodash"
   - Hoisted from "archiver#archiver-utils#lodash"

[kichik]

eslint is a dev dependency and we don't actually use archiver. It might be possible to get rid of the full lodash dependency, at least when using the plugin and not developing. Nothing in serverless itself seems to be using it, so that should actually help installation size and speed.

[dschep]

Ah, didn't realize archiver wasn't actually being used. In that case I definitely prefer just removing the archiver dependency and sticking with the modular imports as is.

[kichik]

graceful-fs can also be removed.

[abetomo]

Thank you for review!
I will check the dependency again.