search

angelnikolov / ts-cacheable

Observable/Promise Cache Decorator
https://npmjs.com/package/ts-cacheable
ISC License
340 stars 42 forks source link

Expose context to storage strategy methods. #105

Closed angelnikolov closed 3 years ago

angelnikolov commented 3 years ago

Fixes #102 @KG3RK3N Will this work for you?

codecov[bot] commented 3 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@f48b15e). Click here to learn what that means. The diff coverage is 57.14%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #105   +/-   ##
=========================================
  Coverage          ?   75.80%           
=========================================
  Files             ?       12           
  Lines             ?      310           
  Branches          ?       80           
=========================================
  Hits              ?      235           
  Misses            ?       60           
  Partials          ?       15
Impacted Files Coverage Δ
common/IAsyncStorageStrategy.ts 100.00% <ø> (ø)
common/IStorageStrategy.ts 100.00% <ø> (ø)
cacheable.decorator.ts 93.33% <50.00%> (ø)
promise.cacheable.decorator.ts 90.00% <50.00%> (ø)
common/InMemoryStorageStrategy.ts 86.36% <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 f48b15e...11c4afb. Read the comment docs.

KG3RK3N commented 3 years ago

Yes, it works fine 👍