Repository navigation
Drop the Stale Private-Repository Note From PhotoCleaner's Registry Entry - #2234
Conversation
…ntry The repository is public, so the note that it is private for now and going public contradicts the live visibility. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Repository: ptr727/ProjectTemplate/.coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID:
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. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #2234 +/- ##
==========================================
Coverage ? 56.61%
==========================================
Files ? 16
Lines ? 7482
Branches ? 0
==========================================
Hits ? 4236
Misses ? 3246
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
It is a single, well-scoped removal of a verified-stale prose note that keeps the JSON valid and leaves no contradicting references in the registry, spec, docs, or TODO.
Review effort: Balanced
Findings: None
What changed in this PR
This PR removes a stale driftNotes entry from the PhotoCleaner repository record in registry/repos.json. The note asserted the repository was "Private for now" and that GitHub-sourced shields would therefore render broken, but the repository is now public (live check: visibility PUBLIC), making the note inaccurate and contradictory to a clean audit. The registry is treated as ground truth about actual repository state, so pruning the obsolete prose keeps the catalog honest.
Changes:
- Deletes the "Private for now by maintainer decision..." sentence from
PhotoCleaner'sdriftNotes. - Leaves the still-accurate multi-arch Docker / two-phase GitHub release note in place.
I verified that registry/repos.json still parses as valid JSON, that the remaining driftNotes is a non-empty array of non-empty strings (satisfying spec/validate.py), and that no other file in registry, spec, docs, or TODO.md still describes PhotoCleaner as private. The only lingering "private" reference is a point-in-time, generated snapshot under reports/, which is out of scope for this change.
| File | Description |
|---|---|
| registry/repos.json | Removes the obsolete private-repository driftNote from PhotoCleaner's entry, leaving the accurate release-shape note intact. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
## Summary Promotes develop to main, carrying the pull requests below, each already reviewed and merged into develop. - [#2203](#2203): Strip every heredoc body a line opens in the wait-loop rule. - [#2216](#2216): Point the charset-unknown message at where a tier is classified. - [#2218](#2218): Name the leftover old tree when the Bash loader cannot remove it. - [#2227](#2227): End the guard's stdin redirect scan at a trailing comment. - [#2230](#2230): Tell apart the causes a labels payload is refused for. - [#2232](#2232): Correct the task names and pointers in the Python tasks snippet header. - [#2234](#2234): Drop the stale private-repository note from PhotoCleaner's registry entry. - [#2237](#2237): State the bootstrap's archive live channel in two skills. - [#2239](#2239): Stop the promotion count on a failed fetch in backlog-burndown. - [#2241](#2241): Render the include walk once per `build_dist.py --check` run. - [#2243](#2243): Have the tree check call `escapes_repo_root` and refuse a symlinked component. - [#2247](#2247): Refuse a Windows drive component anywhere in a tree path. - [#2222](#2222): Tick the adopted repos in the merge-bot and gate rollout stages. - [#2121](#2121): Refuse a dangling remote-tracking target in `local_review`. - [#2249](#2249): Ignore a quoted separator when the guard finds a loop's `done`. Closes #2112 Closes #2171 Closes #1790 Closes #2152 Closes #1372 Closes #2149 Closes #1082 Closes #1772 Closes #1310 Closes #1379 Closes #1452 Closes #2244 Closes #2168 Closes #2106 Closes #2207 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Removes the "Private for now..." sentence from PhotoCleaner's driftNotes, since the repository is public (live check: visibility PUBLIC). The multi-arch Docker sentence is unchanged. No other file in registry, spec, docs, or TODO.md describes PhotoCleaner as private.
Closes on promotion: #1082
🤖 Generated with Claude Code