TimonKK / clickhouse

NodeJS client for ClickHouse
Apache License 2.0
220 stars 122 forks source link

refactor: remove `lodash` #126

Closed 0ss closed 1 year ago

0ss commented 2 years ago

Context

I have noticed that lodash is being used in the client and it's making the client unnecessarily heavy because of it's size

image

so I removed in index.js all the code that uses lodash and it's functions and replaced it with normal JavaScript to exclude lodash.

💡 I haven't removed lodash from dependencies because the test files still use it

TimonKK commented 1 year ago

Could you remove lodash from package.json?