open-duelyst/duelyst

[P3] Replace node-redis with ioredis

开放

#80 创建于 2022年10月2日

在 GitHub 查看贡献者指南
 (0 条评论) (0 个反应) (0 位负责人)JavaScript (526 个派生)batch import
backendenhancementhelp wanted

仓库指标

星标
 (3,443 个星标)
PR 合并指标
 (30 天内没有已合并 PR)

描述

We're currently using node-redis@2.8.0, which has poor support for connection timeouts.

Replacing this with ioredis@5.2.3 fixes the above, but requires some changes. For example, the zrevrankAsync function does not exist, so we need to use zrevrank instead and remove the .then(result) chain.

Alternatively, upgrading node-redis may help with these issues.

See https://github.com/luin/ioredis/wiki/Migrating-from-node_redis

贡献者指南

研究方向
请查阅迁移指南 https://github.com/luin/ioredis/wiki/Migrating from node redis。在代码库中将 node redis 替换为 ioredis，特别是将 `zrevrankAsync` 等方法更新为 `zrevrank` 并移除 `.then()` 链。检查所有 Redis 调用以确保兼容性。
技术栈
javascriptnodejsredis
领域
backend
议题类型
重构
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
半天
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
JavaScriptNode.jsRedis
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
60