feathers icon indicating copy to clipboard operation
feathers copied to clipboard

refactor: reduce usage of lodash

Open fratzinger opened this issue 1 year ago • 1 comments

  • import from lodash submodules directly where possible
  • use const { provider, ...rest } = params instead of omit
  • use { ...params, query: { ...params.query, provider } } instead of merge
  • rm useless flatten
  • use .filter(Boolean) instead of compact(). could be improved with a typesafe IsNonNullable function
  • use () => {} instead of noop

fratzinger avatar Apr 03 '24 12:04 fratzinger

Deploying feathers-dove with  Cloudflare Pages  Cloudflare Pages

Latest commit: 10b7696
Status: ✅  Deploy successful!
Preview URL: https://4a45c9ff.feathers.pages.dev
Branch Preview URL: https://refactor-reduce-lodash-usage.feathers.pages.dev

View logs

Great, I've been meaning to look into this so thank you for doing that.

daffl avatar May 03 '24 16:05 daffl