[ptr727]

Summary

Reconciles the hub's record of HolidayLights with what its main holds since ptr727/HolidayLights#17 promoted the 2025 and 2026 shows and the fleet baseline. This is the registry step of a HolidayLights resync, audit run 2026-11-11T00:50:25Z | hub 1397e30a | branch override develop.

• registry/repos.json. Deletes the driftNote saying the 2025 and 2026 show folders and the baseline (ptr727/HolidayLights#11, Update branding and documentation consistency #13-Enforce squash-only merge strategy in GitHub ruleset and workflow #16) wait on develop for a promotion. The audit flagged it as a note asserting pending work.
• docs/reusable-workflows.md. Ticks HolidayLights in both the merge-bot and the validate-task.yml rows, since its callers have called each by pin since ptr727/HolidayLights#15.
• docs/eol-lf-rollout.md. Ticks HolidayLights as converted in ptr727/HolidayLights#10, and records its 2026-10-24 reclassification to operational.

Verification

• spec/validate.py passes, and the registry parses.
• A read of HolidayLights origin/main confirms each claim: the 2021-2026 show folders, the carried instruction set, a test-pull-request.yml calling validate-task.yml by pin, and * text=auto eol=lf with xlights_effectpresets.json pinned CRLF.
• Recorded local strict review passes cover the head, and the diff-scoped prose gate is clean.

🤖 Generated with Claude Code

Summary by CodeRabbit

• Documentation
  • Updated rollout guidance to reflect the repository’s LF line-ending configuration and adoption of merge-bot and validation workflows.
  • Clarified that the effect presets file uses CRLF line endings.
  • Removed an outdated note about show folders and fleet baseline files awaiting promotion.

[coderabbitai]

No actionable comments were generated in the recent review.

Walkthrough

The rollout checklist now records HolidayLights’ line-ending setup and adoption of two workflows. The repository registry no longer includes a drift note about show folders and the fleet baseline pending promotion.

Changes: HolidayLights rollout

Layer / File(s)	Summary
Line-ending rollout status docs/eol-lf-rollout.md	The checklist marks HolidayLights operational with lineEndings: lf. It documents the conversion and a CRLF pin for xlights_effectpresets.json.
Workflow adoption and drift status docs/reusable-workflows.md, registry/repos.json	The checklist records adoption of the merge-bot and validation workflows on develop and main. The registry no longer includes the note about show folders and the fleet baseline pending promotion.

Estimated code review effort: 2 (Simple) | ~8 minutes

Merge Risk: Minimal · up to ccf91

The changes reconcile rollout records. No merge-blocking issue is established, though the external repository status remains independently unverified.

Pre-merge checks: 5

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title clearly and concisely describes reconciling the HolidayLights registry and rollout trackers after promotion, matching the main changes.
Docstring Coverage		No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)

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

• X
• Mastodon
• Reddit
• LinkedIn

Recent review info

Run configuration

• Source mode: Auto
• Repository: ptr727/ProjectTemplate
• Branch: registry/holidaylights-drift-20261110
• File: .coderabbit.yaml
• Commit: ccf9166
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 6406b26b-3e06-4cd3-9c5b-18eeaaf79b88

Commits

Reviewing files that changed from the base of the PR and between 3416f61 and ccf9166.

Files selected for processing: 3

• docs/eol-lf-rollout.md
• docs/reusable-workflows.md
• registry/repos.json

Files with no reviewable changes: 1

• registry/repos.json

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

[ptr727]

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

[Copilot]

🔵 Needs a closer look

The registry and rollout claims rely on live ptr727/HolidayLights main state, but the configured GitHub API returned 404, preventing independent verification of those cross-repository facts.

No open issues 🎉

What changed in this PR

This pull request reconciles the HolidayLights registry and rollout trackers with its promoted main state.

Changes:

• Removes the obsolete registry note describing pending HolidayLights promotion work.
• Marks HolidayLights as adopted for the merge-bot and validation reusable workflows.
• Records its operational classification and LF conversion.
• Requires regeneration of the workflow reuse report to reflect these adoptions.

File	Description
registry/​repos.json	Removes the outdated drift note describing pending HolidayLights promotion work.
docs/​reusable-workflows.md	Marks HolidayLights as adopted for the merge-bot and validation reusable workflows.
docs/​eol-lf-rollout.md	Records HolidayLights' completed LF conversion and operational classification.

_{Review effort: Lite}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.24%. Comparing base (3416f61) to head (ccf9166).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #4055   +/-   ##
========================================
  Coverage    79.24%   79.24%           
========================================
  Files           38       38           
  Lines        12802    12802           
========================================
  Hits         10145    10145           
  Misses        2657     2657           

Flag	Coverage Δ
python-3.13	79.24% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[ptr727]

@coderabbitai review

[ptr727]

/review

[coderabbitai]

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]

Answering the round on ccf91667 (verdict headline, 0 counted findings): "The registry and rollout claims rely on live ptr727/HolidayLights main state, but the configured GitHub API returned 404, preventing independent verification of those cross-repository facts."

No change needed. ptr727/HolidayLights is private, so the reviewer's token reads a 404 there. The claims were checked against a fetched clone of its main (e226a9e) by read-only git, and by three local strict review passes that re-ran those reads:

• git ls-tree -d origin/main lists 2021 through 2026, and the log holds ptr727/HolidayLights#11 and Update branding and documentation consistency #13-Enforce squash-only merge strategy in GitHub ruleset and workflow #16 merged through promotion Improve README for clarity and consistency in setup instructions #17.
• origin/main:.github/workflows/test-pull-request.yml and merge-bot-pull-request.yml call validate-task.yml and merge-bot-task.yml by pin, both first landed in Improve setup instructions and documentation consistency #15 at 28a714c.
• origin/main:.gitattributes sets * text=auto eol=lf with xlights_effectpresets.json pinned CRLF, from Update readme and workspace #10 at ad734b4. Hub ccc5db3e (Reclassify HolidayLights as an Operational Config Repo #3453) is the 2026-10-24 reclassification to operational.