chore(assets): add 4 missing NCAA team logos (march-madness) - #415
Conversation
…, LEHIGH, MICHIGAN, RUTGERS) march-madness (already live in ledmatrix-plugins) loads team logos from this shared assets/sports/ncaa_logos/<ABBR>.png cache at runtime (manager.py:233) -- these 4 were missing. Split out of PR #412 (chore/dead-code-removal), which had accidentally bundled these in alongside unrelated dead-code deletions. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KEZK1P1Q1fu5pcuVrkrCFZ
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (4)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
Up to standards ✅🟢 Issues
|
Summary
Split out of PR #412 (
chore/dead-code-removal).march-madnessis already live inledmatrix-plugins(version 1.0.1, ahead of the stale local copy that got accidentally committed to core in #412 -- nothing else needed there). It loads team logos at runtime fromassets/sports/ncaa_logos/<ABBR>.png(manager.py:233), and these 4 teams (COLGATE, LEHIGH, MICHIGAN, RUTGERS) were missing from that shared cache.No code changes -- just the 4 binary assets.
🤖 Generated with Claude Code
https://claude.ai/code/session_01KEZK1P1Q1fu5pcuVrkrCFZ