Skip to content

Consider changing Merge / MergeDeep dependency #32

@joshkel

Description

@joshkel

I'm interested in ways of reducing the size of my node_modules directory and the number of dependencies that my projects pull in. I notice that object-deep-merge pulls in type-fest@4.2.0 "as a convenience" but doesn't use it itself. As a result, any projects that use any packages that depend on object-deep-merge get a 552 kB dependency installed, even though it's not needed.

Could you please consider changing this, to reduce dependencies?

  • Drop the type-fest dependency and Merge / MergeDeep types. Update the README to recommend using type-fest directly. (I realize this would be a breaking change.)
  • Alternatively, the type-fest README says that developers can "copy-paste the needed types. No credit required. 👌" So object-deep-merge could copy and paste just those two types. (However, MergeDeep's implementation is fairly involved.)

If neither of these changes fits the direction / goals of object-deep-merge, I understand.

Thank you.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions