Issue 403

[jaytist]

Fixes #404

• Please check if the PR fulfills these requirements

• The commit message follows our guidelines: https://github.com/jaredwray/keyv/blob/main/CODE_OF_CONDUCT.md
• Tests for the changes have been added (for bug fixes/features) with 100% code coverage.
• Docs have been added / updated (for bug fixes / features)

• What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

• Bug fix

• What is the current behavior? (You can also link to an open issue here)

• The Keyv.Options type requires a type of Store, but all stores in this library appear to be implementing Store<string | undefined>. If the Value type is anything but string then using this option isn't possible, typescript will complain. This should be expecting a Store<string | undefined>, which also makes sense considering all values are, by default, serialized to string using JSONB before being passed down to the store.

• What is the new behavior (if this is a feature change)?

• Instead of expecting a Store<string | undefined> , expect Store

• Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

• No

• Other information:

[codecov]

Codecov Report

Merging #417 (60b1526) into main (4efe8e0) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #417   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           18        18           
  Lines         1115      1113    -2     
=========================================
- Hits          1115      1113    -2     

Impacted Files	Coverage Δ
packages/redis/src/index.js	100.00% <ø> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4efe8e0...60b1526. Read the comment docs.

[jaredwray]

@alphmth - I am going to merge the pull request #405

can you update the rest of the packages and testing in this pull request?

[jaredwray]

can you update the dependencies across all of these?