Skip to content

docs: CI/tests 徽章指向修正——test.yml 已改名 ci.yml - #84

Merged
modusensus merged 1 commit into
mainfrom
docs/fix-ci-badge
Sep 8, 2026
Merged

docs: CI/tests 徽章指向修正——test.yml 已改名 ci.yml#84
modusensus merged 1 commit into
mainfrom
docs/fix-ci-badge

Conversation

@modusensus

@modusensus modusensus commented Sep 8, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Member

问题

README 的 CI 徽章指向 test.yml，但该 workflow 已在 #75 改名为 ci.yml，shields 显示 repo and workflow not found。

改动

  • 根 README.md + dsh-mneme/README.md：CI badge 由 test.yml → ci.yml
  • 根 README.md：tests badge 由过期的 815 → 617（与子 README 及实测一致）

验证

新 badge 实时返回：CI build: passing / tests 617 passed。

Summary by CodeRabbit

  • Documentation
    • Updated README status badges to reference the current CI workflow.
    • Updated the documented passing test count to 617.

@modusensus @Anans-Amarre
docs: CI/tests 徽章指向修正——test.yml 已改名 ci.yml，badge 不再 repo and workflow…
686ae45 
… not found

#75 将 CI workflow 由 test.yml 改名为 ci.yml 时漏同步 README 徽章 URL，
两份 README 的 CI badge 均指向不存在的 test.yml（shields 显示
"repo and workflow not found"）；一并把根 README 的 tests 徽章从过期的
815 改为 617（子 README 早已同步）。

Co-Authored-By: Anans-Ivresse <[email protected]>
@coderabbitai

coderabbitai Bot commented Sep 8, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 90d7cae3-4291-418e-84cb-e5513cf70f8c

📥 Commits

Reviewing files that changed from the base of the PR and between f45d090 and 686ae45.

📒 Files selected for processing (2)
  • README.md
  • dsh-mneme/README.md

Included review availability: Your plan provides up to 10 included reviews per hour; 8 remain after this review.

📝 Walkthrough

Walkthrough

The README files now reference the ci.yml workflow. The root README reports 617 passing tests instead of 815.

Changes

Documentation badge updates

Layer / File(s) Summary
Update README badges
README.md, dsh-mneme/README.md		 CI badges now reference the ci.yml workflow. The root README reports 617 passing tests.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 686ae

The README badges now point to the active CI workflow and the displayed test count is updated. No current merge-readiness risk remains.

Suggested reviewers: anans-ivresse

🚥 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 The title clearly describes the primary change: correcting CI and test badge references after renaming the workflow from test.yml to ci.yml.
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)
  • Create PR with unit tests
  • Commit unit tests in branch docs/fix-ci-badge

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

@codecov

codecov Bot commented Sep 8, 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 f907d5a into main Sep 8, 2026
6 checks passed
@modusensus
modusensus deleted the docs/fix-ci-badge branch September 8, 2026 01:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@modusensus