Repository navigation
Describe DevKitCIoT's Registry Notes After Its Promotion - #3927
Conversation
DevKitCIoT#20 promoted the validate-task caller, the project-local lib/ and the hub standup to main, so the notes no longer call them pending. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
A recorded local strict-review pass covers head |
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the
|
There was a problem hiding this comment.
🟡 Changes recommended
An active nit remains because registry/repos.json still labels a resolved promotion as current drift, contrary to the repository’s audit guidance.
1 open issue
What changed in this PR
Updates DevKitCIoT’s registry metadata to reflect its promotion to main and document the repository’s current validation and library state.
Changes:
- Updates stale promotion-related
driftNoteswording. - Documents the
validate-taskintegration and KiCad ERC/DRC checks. - Documents the project-local KiCad parts in
lib/. python3 spec/validate.pypasses.
| File | Review summary |
|---|---|
registry/repos.json |
Refreshes DevKitCIoT’s registry notes. One nit requests removing the completed-promotion note because driftNotes should represent current deviations rather than historical work. |
This pull request is ready to be approved.
Review effort: Lite
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #3927 +/- ##
========================================
Coverage 78.26% 78.26%
========================================
Files 35 35
Lines 12328 12328
========================================
Hits 9648 9648
Misses 2680 2680
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:
|
|
@coderabbitai review |
|
/review |
Action not completed
|
A driftNote records a current deviation, and a resolved one is deleted rather than rewritten as history (AUDIT.md). So drop the completed promotion note and the baseline validate-task caller. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The ERC/DRC hook meets the eda validation check rather than deviating from it, so its sentence goes. The lib/ note now states only what lasts: the parts are a frozen copy that later library releases do not reach. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
A recorded local strict-review pass covers head |
…s and Refresh DevKitCIoT (#3926) ## Summary Promotes #3923, #3924 and #3927 to `main`. - **Three repositories move their Copilot and CodeRabbit reviews to explicit requests (#3924).** PlexCleaner, Financial-Modeling and HomeAutomation-Config each get the hub's own values: Copilot explicit, after-checks, at most 3 rounds, gating; CodeRabbit explicit, after-checks, once. Qodo keeps the fleet default. This is a test of explicit review on active work, from the review-cutover track (#3877). - **DevKitCIoT's registry entry is refreshed (#3923).** It drops `pullRequestCheck: false`, so `configure.sh apply` adds DevKitCIoT's required status check, and it adds the repository description. - **DevKitCIoT's registry notes keep only its current deviations (#3927).** This answers Copilot's finding on this promotion. The completed-promotion text is deleted, per `AUDIT.md`'s driftNotes rule. Nothing is written to any repository by this promotion. Each of the three repositories' own sessions writes its policy files into its `develop`. `configure.sh apply` then runs for each from a hub checkout at the new `main`, with its own go-ahead. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

Rewrites DevKitCIoT's registry
driftNotes, which still called its promotion pending after DevKitCIoT#20 merged it intomain. This answers Copilot's finding on promotion #3926.Each new claim was checked against DevKitCIoT
main:.github/workflows/test-pull-request.ymlcalls the hub'svalidate-task.yml..github/actions/validate/action.ymlruns the kicad-cli ERC and DRC checks.lib/holds the parts copied from the 2.0.0 library release.main.spec/validate.pypasses.Pre-existing findings, not filed
The local review also flagged rollout-tracker lines that went stale at the same promotion. This change does not touch them.
docs/reusable-workflows.md:181: DevKitCIoT's validate-task caller box is unchecked, butmainnow carries the caller. Class: pre-existing.docs/reusable-workflows.md:149: DevKitCIoT is listed as carrying no merge-bot, butmainnow carriesmerge-bot-pull-request.yml. Class: pre-existing.docs/eol-lf-rollout.md:127: the entry describes DevKitCIoTmainas CRLF, butmainnow sets LF in.editorconfigand.gitattributes. Class: pre-existing.Dropped findings
registry/repos.json:264: the reviewer asked whetherstatusshould readarchived, citing DevKitCIoT's README.gh api repos/ptr727/DevKitCIoTreadsarchived=false, socatalogedis correct.🤖 Generated with Claude Code