Skip to content

WLED Org #1994

Open
netmindz wants to merge 6 commits intofrenck:mainfrom
netmindz:wled-dev
Open

WLED Org #1994
netmindz wants to merge 6 commits intofrenck:mainfrom
netmindz:wled-dev

Conversation

@netmindz
Copy link
Copy Markdown

@netmindz netmindz commented Mar 13, 2026

WLED is no longer a personal project, but has moved to the WLED org

(You can verify this claim by going to the old address and you will see the redirect to this one)

Summary by CodeRabbit

  • Chores
    • Updated release retrieval mechanism to fetch updates from the wled/WLED GitHub repository.
    • Modified upgrade flow and API endpoints to reference the new repository location.
    • Updated internal documentation and issue references accordingly.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 13, 2026

📝 Walkthrough

Walkthrough

Updated GitHub repository references from Aircoookie/WLED to wled/WLED in the CLI and core upgrade components. Release links and API endpoints now point to the new repository location.

Changes

Cohort / File(s) Summary
Repository Reference Updates
src/wled/cli/__init__.py, src/wled/wled.py
Updated GitHub repository URLs in release links, upgrade flow, and API endpoints from Aircoookie/WLED to wled/WLED. Changes are literal URL replacements with no logic or control flow modifications.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 From Aircoookie's nest to WLED's bright home,
The URLs now clearly show where to roam,
Updated paths for updates and releases true,
Hopping forward with the repository's new crew! 🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'WLED Org' is vague and does not clearly convey the specific changes made—migrating repository references from Aircoookie/WLED to wled/WLED. Use a more descriptive title like 'Migrate repository URLs from Aircoookie/WLED to wled/WLED' to clearly communicate the primary change to reviewers.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📝 Coding Plan
  • Generate coding plan for human review comments

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

Comment @coderabbitai help to get the list of available commands and usage tips.

Tip

You can disable sequence diagrams in the walkthrough.

Disable the reviews.sequence_diagrams setting to disable sequence diagrams in the walkthrough.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
src/wled/wled.py (1)

631-631: Consider cleaning up the remaining legacy repo reference for consistency.

Since this PR standardizes repository references, you may also want to update the old Aircoookie/WLED issue URL comment in src/wled/models.py (Lines 752-755) to avoid mixed historical links.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@src/wled/wled.py` at line 631, Replace the lingering legacy repository
reference in the comment inside the wled.models module that currently points to
"Aircoookie/WLED" (the comment referencing the issue URL) with the standardized
repository reference "WLED/WLED" (e.g., update the URL to
https://github.com/WLED/WLED/issues/3257); locate the comment in the models
module that contains the text "Aircoookie/WLED" and update the text and URL for
consistency, and scan nearby comments in the same module for any other legacy
references and change them to the new repo format.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@src/wled/wled.py`:
- Line 631: Replace the lingering legacy repository reference in the comment
inside the wled.models module that currently points to "Aircoookie/WLED" (the
comment referencing the issue URL) with the standardized repository reference
"WLED/WLED" (e.g., update the URL to https://github.com/WLED/WLED/issues/3257);
locate the comment in the models module that contains the text "Aircoookie/WLED"
and update the text and URL for consistency, and scan nearby comments in the
same module for any other legacy references and change them to the new repo
format.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6681272d-ea95-4d75-9201-6b66c050fa7f

📥 Commits

Reviewing files that changed from the base of the PR and between 367b493 and f8b77bb.

📒 Files selected for processing (2)
  • src/wled/cli/__init__.py
  • src/wled/wled.py

@github-actions
Copy link
Copy Markdown

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions.

@github-actions github-actions bot added the stale There has not been activity on this issue or PR for quite some time. label Apr 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale There has not been activity on this issue or PR for quite some time.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant