node-redis icon indicating copy to clipboard operation
node-redis copied to clipboard

[SUGGESTION] Cleaned up search index & json module

Open Didas-git opened this issue 3 years ago • 1 comments

Description

Since i commented about this I decided to take a look and cleanup the file a bit. This should be taken with a grain of salt bcz its purelly based on my opinion.

Also implemented all the changes in #1901 and the suggestion i commented about, keyof any to PropertyKey


Checklist

  • [x] Does npm test pass with this change (including linting)?
  • [x] Is the new or changed code fully tested?
  • [x] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

Didas-git avatar Feb 09 '22 17:02 Didas-git

I also thought about changing some of the if statements to &&

Didas-git avatar Feb 09 '22 18:02 Didas-git