fix: reduce usage of lodash

feathersjs / feathers

The API and real-time application framework

https://feathersjs.com

MIT License

14.97k stars 742 forks source link

Closed fratzinger closed 1 month ago

fratzinger commented 2 months ago

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

cloudflare-pages[bot] commented 2 months ago

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

daffl commented 1 month ago

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