axios-cache-adapter icon indicating copy to clipboard operation
axios-cache-adapter copied to clipboard

Remove `constructor.name` assertion

Open OiYouYeahYou opened this issue 3 years ago • 4 comments

This breaks current compatibility with the current redis version and causes an unreasonable expectation of future changes in the future

Fixes: #263

OiYouYeahYou avatar Dec 23 '21 17:12 OiYouYeahYou

Codecov Report

Merging #264 (19e7cfb) into master (2d51cee) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #264   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           13        13           
  Lines          235       229    -6     
=========================================
- Hits           235       229    -6     
Impacted Files Coverage Δ
src/redis.js 100.00% <ø> (ø)
src/serialize.js 100.00% <0.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 2d51cee...19e7cfb. Read the comment docs.

codecov[bot] avatar Dec 23 '21 18:12 codecov[bot]

When are you guys planning to fix this issue, its still breaking while connecting to Redis

anish-joy avatar Mar 11 '22 15:03 anish-joy

@anish-joy unfortunately I reached out to @RasCarlito just before Christmas last year but haven't heard anything back. At this point I think it's fair to declare this as an abandoned project 😔

#265 has an alternative listed

OiYouYeahYou avatar Apr 11 '22 13:04 OiYouYeahYou

@OiYouYeahYou Thanks for the update! I really appreciate it

anish-joy avatar Apr 11 '22 15:04 anish-joy