Skip to content

docs: 双 README 重复徽章行去重 - #319

Merged
modusensus merged 1 commit into
mainfrom
fix/readme-dup-badges
Sep 26, 2026
Merged

modusensus merged 1 commit into
mainfrom
fix/readme-dup-badges

Conversation

@modusensus

@modusensus modusensus commented Sep 26, 2026 •

Copy link
Copy Markdown
Collaborator

9-24 的合并提交 37e77f1(解 #275 车道与 main 的冲突)把 tests 徽章行与中英 npm test 命令注释各复制了一份——根 README 三处(徽章 + 两个语言段的命令注释)、包内 README 两处。

badge:sync 的全文正则替换只会把重复行一起刷新成新数字、永不自愈,故一直留存至今。本 PR 手工去重(各留一行),CHANGELOG [Unreleased] 🧹 小节登记。纯文档改动,零代码影响。

Summary by CodeRabbit

  • 文档
    • 清理中英文 README 中重复的测试数量徽章及开发命令示例中的测试相关内容,每项仅保留一处或不再展示。
    • 在更新日志中记录此次去重,并说明徽章同步操作不会自动移除重复内容。

9-24 合并提交 37e77f1 解冲突时把 tests 徽章行与中英 npm test
命令注释各复制了一份(根 README 三处、包内 README 两处重复)。
badge:sync 的全文正则替换只会把重复行一起刷新、永不自愈,
此番手工去重各留一行;CHANGELOG [Unreleased] 登记。
Copilot AI lite review requested due to automatic review settings September 26, 2026 12:08

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@coderabbitai

coderabbitai Bot commented Sep 26, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: slow-stack/mneme/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: c3373e88-57b1-48fb-b690-e6af2e3737d6

📥 Commits

Reviewing files that changed from the base of the PR and between 6c5adc0 and 7ee7d36.

📒 Files selected for processing (3)
  • README.md
  • dsh-mneme/CHANGELOG.md
  • dsh-mneme/README.md
💤 Files with no reviewable changes (2)
  • README.md
  • dsh-mneme/README.md

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.


📝 Walkthrough

Walkthrough

根 README 和包内 README 删除了重复的测试徽章、测试数量说明或测试命令。变更日志记录了此次清理,并说明 badge:sync 不会自动删除重复行。

Changes

README 测试信息去重

Layer / File(s) Summary
清理 README 测试信息
README.md, dsh-mneme/README.md, dsh-mneme/CHANGELOG.md
根 README 删除首页测试徽章及中英文开发命令中的测试数量说明。包内 README 删除重复徽章和开发命令中的 npm test 行。变更日志记录了去重情况,并说明 badge:sync 不会自动删除重复行。

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Other

Suggested reviewers: heptaspirit

Merge Risk: ⚪ Minimal · up to 7ee7d

这项文档清理未显示会影响测试或其他工作流程的风险,可以合并。

Architecture Summary

Architecture risk: 🔵 Low · up to 7ee7d

The change affects 2 systems.

Changed systems: dsh-mneme, README.md

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — dsh-mneme (service) was modified; 2 changed files map to changed impact.
  • observed — README.md (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in README.md: 移除了首页的测试数量徽章。
  • observed — Modified behavior in README.md: 移除了本地开发命令中 npm test 后的测试数量注释。
  • observed — Modified behavior in README.md: 移除了英文本地开发命令中 npm test 后的测试数量注释。
  • observed — Modified behavior in dsh-mneme/CHANGELOG.md: 在未发布版本的「清理」部分新增 README 去重记录,列明根 README 三处及包内 README 两处重复的 tests 徽章与 npm test 注释已各留一行;原有文字指出 badge:sync 的全文替换不会自行去重。
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 标题准确概括了根 README 和包内 README 的重复徽章去重变更,与 PR 的主要目标一致。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Sep 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@modusensus
modusensus merged commit 00bf750 into main Sep 26, 2026
9 checks passed
@modusensus
modusensus deleted the fix/readme-dup-badges branch September 26, 2026 12:22
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