Skip to content

docs: 移除 README 中已废弃的 redis 服务引用#28

Merged
longsizhuo merged 1 commit intoInvolutionHell:mainfrom
F1sh2712:docs/remove-redis-reference
May 10, 2026
Merged

docs: 移除 README 中已废弃的 redis 服务引用#28
longsizhuo merged 1 commit intoInvolutionHell:mainfrom
F1sh2712:docs/remove-redis-reference

Conversation

@F1sh2712
Copy link
Copy Markdown
Contributor

@F1sh2712 F1sh2712 commented May 8, 2026

What

移除 README 中对已废弃 redis 服务的引用,共 4 处。

Why

docker-compose.yml 中已不存在 redis 服务,但 README 仍指示用户执行
docker compose up -d postgres redis,导致新贡献者运行时报 no such service: redis

Refs #25

How

修改 README.md 共 4 处:

  • 技术栈列表:删除 Redis 7,
  • 目录结构注释:docker-compose.yml # 中间件编排 (Postgres, Redis, Caddy, Backend)docker-compose.yml # 中间件编排 (Postgres, Caddy, Backend)
  • 启动命令注释:启动 PostgreSQL 18 与 Redis 7启动 PostgreSQL 18
  • 启动命令本身:docker compose up -d postgres redisdocker compose up -d postgres

Testing

文档改动,无需代码测试。本地已验证修改后命令可正常执行。

Copy link
Copy Markdown
Member

@longsizhuo longsizhuo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM,本地切到这个分支扫了一遍 README 和 docker-compose.yml 里的 redis 字符串都没了。

仓库里其它地方还有几处历史残留(不在这个 PR 范围,可以下次顺手收一下,或者单开一个 follow-up issue):

  • pom.xml:138-141, 198-201 注释掉的 spring-boot-starter-session-data-redis-test 依赖
  • src/test/resources/application-test.properties:14management.health.redis.enabled=false(没装 redis 写不写都不影响,但语义上已经 dead)
  • src/main/java/com/involutionhell/backend/community/service/SharedLinkService.java:132 javadoc 里把 "Redis INCR + EXPIRE" 当限流示例,这个其实不算错(只是举例),保留也行

文档侧本 PR 干净利落,可以合。

Copy link
Copy Markdown
Member

@longsizhuo longsizhuo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

无需改动,可以合。pom.xml/javadoc/test-properties 里的 redis 残留不在本 PR 范围,后续单开 issue 收。

@longsizhuo longsizhuo merged commit 48853dd into InvolutionHell:main May 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants