Skip to content

Move the Codecov Link Into the README External Group - #1260

Merged
ptr727 merged 1 commit into
developfrom
readme-distribution-codecov
Nov 12, 2026
Merged

ptr727 merged 1 commit into
developfrom
readme-distribution-codecov

Conversation

@ptr727

@ptr727 ptr727 commented Nov 12, 2026 •

Copy link
Copy Markdown
Owner

Follow-up to the hub resync in #1259, per the hub's RESYNC.md section 6. It closes the one remaining audit finding.

Audit --branch develop run 2026-11-12T22:45:45Z | hub 009d3e5e reported readme: the Distribution group holds external reference(s). Audit --branch readme-distribution-codecov run 2026-11-12T22:46:43Z | hub 009d3e5e reports 0 findings, with the README drift gone.

Change

  • Moves [coverage-link] (the Codecov dashboard) from the README Distribution group to External, at its sorted position. Under the hub's readme-sections.json distribution.urlPrefixes, only this project's GitHub repository and Docker Hub pages count as distribution, so the Codecov host is external.

Verification

  • Commit hook: markdownlint and cspell clean.
  • Prose gate --diff against the merge base: no violations.
  • Local strict review recorded with 0 findings. The pass checked group placement, sort order, group order and naming, and that the reference is still used.

Remaining Advisories

The remaining audit lines are intent staleness hints and were checked by hand. .editorconfig-checker.json is the known false positive, since its intent reference is hub GOVERNANCE.md. AUDIT.md's hub reference change covers an unregistered repository and an unpromoted registry entry, neither of which applies here.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Reorganized the Codecov coverage link in the README; the link destination remains unchanged.

The Distribution group holds only this project's own presence on a
platform, and the Codecov dashboard is a third-party host.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings November 12, 2026 22:47
@ptr727

ptr727 commented Nov 12, 2026

Copy link
Copy Markdown
Owner Author

A recorded local strict-review pass covers head 5d5afe14c6dd7f3342a80bfce74f95b1f0c0d651, the content this pull request carries at that commit against develop, and it recorded 0 findings.

@coderabbitai

coderabbitai Bot commented Nov 12, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The README moves the Codecov coverage link definition from the distribution references to the external references. The URL is unchanged.

Changes: README link references

Layer / File(s) Summary
Move Codecov reference
README.md
The coverage-link definition moves from the distribution references to the external references. Its URL is unchanged.

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

Change: Other

Possibly related PRs

Merge Risk: Minimal · up to 5d5af

This is a documentation-only reordering of a link reference with no change to behavior or the link target. It is safe to merge.

Pre-merge checks: Passed 4

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title says the Codecov link moves into the README External group. The raw summary confirms the Codecov link definition moved from the distribution references to the external references section, wi…
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.
Full details: Title check

Explanation

The title says the Codecov link moves into the README External group. The raw summary confirms the Codecov link definition moved from the distribution references to the external references section, with its URL unchanged. The title matches the main change, so the check passes.

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

Recent review info

Run configuration

Commits

Reviewing files that changed from the base of the PR and between 9bc274f and 5d5afe1.

Files selected for processing: 1

  • README.md

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

@qodo-free-for-open-source-projects

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can open the rule or relationship behind a finding in one click

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@qodo-free-for-open-source-projects

Copy link
Copy Markdown

PR Summary by Qodo

Move Codecov README Link to External References

📝 Documentation 🕐 Less than 5 minutes

Grey Divider

AI Description

• Moves the Codecov link from Distribution to External, where third-party hosts belong.
• Preserves the coverage badge target while resolving the README section audit finding.
High-Level Assessment

Moving the existing reference is the simplest way to satisfy the section rules without changing the link or its usage.

Files changed (1) +1 / -1

Documentation (1) +1 / -1
README.mdClassify the Codecov link as an external reference +1/-1

Classify the Codecov link as an external reference

• Moves the coverage-link definition from Distribution to its sorted position in External. The URL and coverage badge reference remain unchanged.

README.md

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.

🟢 Approval recommended

The Codecov reference is now in the alphabetically sorted External group while preserving the existing URL and badge usage, with no active review issues identified.

No open issues 🎉

What changed in this PR

Moves the Codecov reference definition from the README's Distribution group to the External group, aligning the README with the hub's link classification rules while preserving the existing coverage badge link.

Changes:

  • Relocated [coverage-link] to the alphabetically sorted External group.
  • Preserved the existing Codecov URL and badge usage.
File Description
README.md Reclassifies the Codecov reference under the README's External link group.

Review effort: Lite

@ptr727

ptr727 commented Nov 12, 2026

Copy link
Copy Markdown
Owner Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Nov 12, 2026 •

Copy link
Copy Markdown
Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@ptr727
ptr727 merged commit 2617989 into develop Nov 12, 2026
16 checks passed
@ptr727
ptr727 deleted the readme-distribution-codecov branch November 12, 2026 22:54
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