Skip to content

Move the status badges to a Status section at the end of the README (issue btclib-org/.github#1856) - #492

Merged
fametrano merged 1 commit into
mainfrom
iss-1856-readme-status
Nov 8, 2026
Merged

fametrano merged 1 commit into
mainfrom
iss-1856-readme-status

Conversation

@fametrano

Copy link
Copy Markdown
Member

The head of README.md carries only what the package is: section 2's software badges, then OpenSSF Best Practices and Baseline. The gates and the sentinels move to a closing ## Status section above the supported-by line, with the Scorecard badge as the last sentinel.

The comment above the badges becomes one line pointing to section 2 of btclib-org/.github's README.md, which holds the rules it restated.

This is one of the fifteen tree ports of section 2's ## Status rule, which PR btclib-org/.github#2105 set. It closes nothing. ISS btclib-org/.github#1856 is closed by hand once every tree has landed and .github's BACKLOG row for 1856 is gone.

Evidence

Check Result
.github's test_a_readme_keeps_its_gates_and_sentinels_in_its_status_section, run from .github origin/main 3a774f4a the tip passes; the base fails ("no ## Status section is the last ## section")
Badges kept the sorted badge lines hash the same at the base and at the tip, with equal counts
Gates on 10c4501, under the gate lock (writer's log) pre-commit run --all-files, the suite and the docs build with -W: exit 0
Merge with origin/main 980b2c6 merge-tree exit 0, no conflict
Local review dedicated reviewer on the port, then a fresh reviewer on the amend: CLEARED 10c4501

The fresh review in the other client did not run. Codex answers: "ERROR: Your workspace is out of credits. Add credits to continue."

🤖 Generated with Claude Code

…issue btclib-org/.github#1856)

The head of README.md carries only what the package is: section 2's
software badges, then OpenSSF Best Practices and Baseline. The gates and
the sentinels move to a closing `## Status` section above the
supported-by line, with the Scorecard badge as the last sentinel.

The comment above the badges becomes one line pointing to section 2 of
btclib-org/.github's README.md, which holds the rules it restated.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Ferdinando Ametrano <ferdinando@ametrano.net>
@coderabbitai

coderabbitai Bot commented Nov 8, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Summary

Summary by CodeRabbit

  • Documentation
    • Added a “Status” section to the README with the workflow and OpenSSF Scorecard badges.
    • Updated badge guidance and clarified the Scorecard badge description.

The README now places workflow and OpenSSF Scorecard badges in a new Status section after Links. It replaces detailed badge-placement comments with a link to the organization’s README guidance. The Scorecard workflow comment now describes the badge’s Status section placement.

Changes: Badge placement

Layer / File(s) Summary
README badge placement
.github/workflows/scorecard.yml, README.md
The README replaces detailed badge comments with a link to badge guidance and moves four badges to a Status section after Links. The Scorecard workflow comment describes the badge’s new location.

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

Change: Other

Suggested labels: github_actions

Merge Risk: Minimal · up to 10c45

The badge reorganization presents no identified merge-blocking risk.

Architecture risk: Low · up to 10c45

The change affects 1 system.

Changed systems: README.md

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — README.md (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in README.md: The detailed comments explaining badge categories, ordering, formatting, and where tooling choices are documented were replaced with a pointer to the organization’s README guidance.
  • observed — Modified behavior in README.md: The pre-commit, workflow, documentation-build, and OpenSSF Scorecard badges were removed from their former position after the package badges.
  • observed — Modified behavior in README.md: A “Status” section was added after “Links” containing the pre-commit, CI workflow, documentation-build, and OpenSSF Scorecard badges removed from the introductory badge area.
  • observed — Modified behavior in .github/workflows/scorecard.yml: The comment changes the badge description from one at the head of the README’s last line to the last badge in the README’s ## Status section; it also describes the Scorecard as published.
Pre-merge checks: Passed 4

Check name Status Explanation
Title check Passed The title "Move the status badges to a Status section at the end of the README (issue btclib-org/.github#1856)" matches the main change: the README's gates and sentinel badges move to a closing `## St…
Description check Passed The description states the purpose of the change: the README head keeps only the software badges plus OpenSSF Best Practices and Baseline, while the gates and the Scorecard sentinel move to a closing …
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 "Move the status badges to a Status section at the end of the README (issue btclib-org/.github#1856)" matches the main change: the README's gates and sentinel badges move to a closing ## Status section, and the Scorecard workflow header comment is updated to match. The title names the issue reference, which helps traceability. The title cannot be checked against the organization's standard in full, because the referenced section 11 text was not retrieved. Under the criteria in this check, the title is related to the main change.

Full details: Description check

Explanation

The description states the purpose of the change: the README head keeps only the software badges plus OpenSSF Best Practices and Baseline, while the gates and the Scorecard sentinel move to a closing ## Status section. It also explains the one-line comment pointer, the related issue and PR, and the verification evidence. It is clearly related to the changeset.

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

Usage-based review receipt

  • Mode: Continue automatically
  • Reviewed files: 2
  • Waived: $0.50 (charged $0.00)
  • View usage details

Note

This review exceeded your plan’s limits and used usage-based reviews—free during trial, billed after paid activation unless disabled. Manage usage-based reviews.

Recent review info

Run configuration

Commits

Reviewing files that changed from the base of the PR and between 980b2c6 and 10c4501.

Files selected for processing: 2

  • .github/workflows/scorecard.yml
  • README.md

Limit details: You’ve used all 10 included reviews currently available.

Comment thread .github/workflows/scorecard.yml Resolved

@claude claude Bot 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.

ACK 10c4501

  • non-blocking: .github/workflows/scorecard.yml:5-6: "the last badge of the README's ## Status section" says the order rule a second time. The badge is last only because scorecard holds the latest slot in section 10's calendar (Sunday 05), and section 2 of btclib-org/.github's README.md already sets that order. A later Sunday sentinel would make the comment wrong, and no test reads it. The standard's section 9, One fact in one place, is the rule. There is a suggestion inline. I leave it to the author.

Not checked: the PR body's .github test test_a_readme_keeps_its_gates_and_sentinels_in_its_status_section and its badge-hash comparison were not run. I have no interpreter.

@fametrano
fametrano merged commit d4b5c0d into main Nov 8, 2026
25 checks passed
@fametrano
fametrano deleted the iss-1856-readme-status branch November 8, 2026 05:27
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.

1 participant