search

permitio / opal

Policy and data administration, distribution, and real-time updates on top of Policy Agents (OPA, Cedar, ...)
https://opal.ac
Apache License 2.0
4.48k stars 163 forks source link

Use redis-py to replace asyncio-redis #498

Closed calmzhu closed 1 year ago

calmzhu commented 1 year ago

Fixes Issue

Close #497

Use redis.asyncio from redis-py to replace asyncio-redis. Since package asyncio-redis had already been integrated to redis-py and abanded

netlify[bot] commented 1 year ago

Deploy Preview for opal-docs canceled.

Name Link
Latest commit 08798da7e4a710ea42b75c85b4ff0c4d887bd9dd
Latest deploy log https://app.netlify.com/sites/opal-docs/deploys/65128245c62036000870ecb7
asafc commented 1 year ago

Hi @calmzhu, thanks for the contribution! It might take us a couple days to review due to workload (it's on @roekatz 's queue, no worries :))

calmzhu commented 1 year ago

Fixed in my local environment. Can you run this workflow again

roekatz commented 1 year ago

Thanks @calmzhu ! 🚀

orweis commented 1 year ago

Thanks @calmzhu 💪

@danielbass37 - please make sure to put him on our swag award list.

danielbass37 commented 1 year ago

@orweis I'd love to! @calmzhu are you in OPAL's Slack community?

calmzhu commented 1 year ago

@orweis I'd love to! @calmzhu are you in OPAL's Slack community?

Yes, i just joined.