Skip to content

Lodash dependency - required? #428

@Daveawb

Description

@Daveawb

Is this a question? Yes and... no? Go figure.

I noticed that I had the full blown version of lodash in my dependencies once I had installed this package (which is excellent btw). I started to look through the code base to see if it really was required and I have only found these usages:

context-matcher.ts
_.isFunction
_.isString

config-factory.ts
_.assign
_.isString
_.isPlainObject

handlers.ts
_.camelCase
_.get
_.isFunction

http-proxy-middleware.ts
_.assign
_.isFunction

path-rewriter.ts
_.isFunction
_.forEach
_.isEmpty
_.isPlainObject
_.isUndefined
_.isNull
_.isEqual
_.isPlainObject
_.forIn

router.ts
_.isPlainObject
_.isFunction
_.forIn

Each one of these dependencies can be achieved easily with ES6 without the need for lodash, would you be willing to receive a PR to remove this dependancy?

Most of my projects are built by CI many times / day and anything I can do to reduce the time needed to bootstrap each run is always appeciated.

Is this a bug report?

No

Is this a feature request?

Sort of. This request is to remove the lodash dependancy from the project.

Motivation:

  • Faster installs. Better performance.

Setup

Not applicable

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions