Promotion prep 2: reconcile groundTruthBranch to main; fix doc drift (#383) - #385
Merged
Conversation
…383) Pre-existing cross-commit drift surfaced by the develop->main promotion review. - registry/repos.json: KiCadLibrary, EspDinIoT, PhotoCleaner, and HolidayLights set groundTruthBranch: develop, which AUDIT.md says is never valid (ground truth is always the promoted main; un-promoted develop work counts when it reaches main). All four have a main branch, so the override was stale - remove it and let them default to main. - docs/repo-config-carry.md: the carried-files bullet banned all cross-repo links, stricter than AGENTS.md, which allows a contextually relevant link; align it. Add configure.sh's payload-inference fallback (registry-absent carries) to the configure step. - STANDUP.md: note the same configure.sh payload-inference fallback. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR prepares for the develop -> main promotion by removing stale groundTruthBranch: develop overrides from the fleet registry (so affected repos default back to main) and by aligning onboarding/carry documentation with the current rules and repo-config/configure.sh behavior.
Changes:
- Remove
groundTruthBranch: developfrom fourregistry/repos.jsonentries so they default tomain. - Update
docs/repo-config-carry.mdto align the carried-files linking rule withAGENTS.mdand documentconfigure.sh's model inference fallback. - Update
STANDUP.mdto mention the sameconfigure.shfallback when no registry is present.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| STANDUP.md | Documents repo-config/configure.sh model selection fallback (registry lookup or carried payload). |
| registry/repos.json | Removes stale per-repo groundTruthBranch: develop overrides so affected repos default to main. |
| docs/repo-config-carry.md | Aligns carried-files link guidance with AGENTS.md and documents configure.sh fallback behavior. |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
…al (#385) KiCadLibrary's note called develop "the recorded ground-truth branch" (the override this PR removed), so drop that phrase. PhotoCleaner's note claimed there is no main branch, but main now exists, so restate it as pre-conformance with the work-in-progress content on develop. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Uh oh!
There was an error while loading. Please reload this page.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Second batch of pre-existing cross-commit drift surfaced by the
develop→mainpromotion review (#383). Fixed ondevelopso the promotion carries clean content.registry/repos.json— KiCadLibrary, EspDinIoT, PhotoCleaner, HolidayLights setgroundTruthBranch: develop, which AUDIT.md says is never valid (ground truth is always the promotedmain; un-promoteddevelopwork "counts when it reachesmain"). All four have amainbranch, so the override was stale — removed, so they default tomain.docs/repo-config-carry.md— the carried-files bullet banned all cross-repo links, stricter than AGENTS.md (which allows a contextually relevant link); aligned. Also addedconfigure.sh's payload-inference fallback (registry-absent carries).STANDUP.md— noted the sameconfigure.shpayload-inference fallback.spec/validate.pygreen (no repo ongroundTruthBranch: develop); markdownlint clean; CRLF preserved.🤖 Generated with Claude Code