Restored redis 4 and master in CI

[WisdomPill]

No description provided.

[codecov]

Codecov Report

Merging #575 (cd43aaa) into master (df9a325) will increase coverage by 0.1%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master    #575     +/-   ##
========================================
+ Coverage    57.7%   57.8%   +0.1%     
========================================
  Files          39      39             
  Lines        2497    2497             
  Branches       63      73     +10     
========================================
+ Hits         1440    1441      +1     
+ Misses       1049    1041      -8     
- Partials        8      15      +7     

Flag	Coverage Δ
mypy	33.9% <ø> (+0.1%)	⬆️
tests	83.3% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
django_redis/cache.py	97.0% <0.0%> (ø)
tests/test_backend.py	14.7% <0.0%> (+0.2%)	⬆️

---

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 df9a325...cd43aaa. Read the comment docs.

[WisdomPill]

@hartwork do you think we could work around the issue with packaging by installing it when using redis-master?

[hartwork]

@hartwork do you think we could work around the issue with packaging by installing it when using redis-master?

@WisdomPill technically yes, but please note #576 (comment) for the bigger picture.

[hartwork]

@WisdomPill Installation of package packaging seems still missing.
PS: My vote for merging this without squashing after that addition.

[hartwork]

@WisdomPill the way how packaging is added right now does not guarantee that it's installed prior to redis-py, which is causing the breakage. If that's not easily fixable, my vote for re-opening #576 and merge that fix in isolation, first.

[WisdomPill]

agreed

[hartwork]

@WisdomPill thank you!