sindresorhus / p-memoize

Memoize promise-returning & async functions
MIT License
396 stars 30 forks source link

Move `type-fest` to development dependencies #54

Closed mahnunchik closed 1 year ago

mahnunchik commented 1 year ago

I think it is better to move type-fest module to development dependencies.

Motivation: it doesn't required in compiled .js file.

And I don't need typescipt in my application in production:

└─┬ p-memoize@7.1.1
  └─┬ type-fest@3.10.0
    └── typescript@5.0.4
sindresorhus commented 1 year ago

The type is used in production, so that would be a mistake. As for the typescript thing, see: https://github.com/sindresorhus/p-memoize

mahnunchik commented 1 year ago

Ok, got it, typescript is cancer on the body of modern js.