Skip to content

Remove duplicate Matter server URL. - #4932

Merged
sairon merged 1 commit into
home-assistant:masterfrom
kylev:kv_url_dupe
Nov 11, 2026
Merged

sairon merged 1 commit into
home-assistant:masterfrom
kylev:kv_url_dupe

Conversation

@kylev

@kylev kylev commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

The one remaining is correct

Summary by CodeRabbit

  • Chores
    • The server configuration now lists only the app-matter-server GitHub URL; the addons repository URL is no longer included.

The one remaining is correct
Copilot AI balanced review requested due to automatic review settings November 11, 2026 01:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No active review issues remain for the Matter Server metadata URL cleanup.

No open issues 🎉

What changed in this PR

Corrects the Matter Server add-on’s metadata URL by removing the extra repository link.

Changes:

  • Keeps https://github.com/home-assistant/app-matter-server as the sole URL.
File Description
matter_server/​config.yaml Removes the extra URL from the add-on metadata.

Review effort: Balanced

@coderabbitai

coderabbitai Bot commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review.

Walkthrough

The Matter Server configuration URL now contains the app-matter-server GitHub URL without the addons repository URL.

Changes: Matter Server configuration

Layer / File(s) Summary
Update configuration URL
matter_server/config.yaml
The URL value retains the app-matter-server GitHub URL and no longer includes the addons repository URL.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Suggested reviewers: homeassistant

Merge Risk: Minimal · up to 723b5

The configuration retains the Matter Server link and removes the duplicate. No merge-blocking risk is established.

Pre-merge checks: Passed 5

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly summarizes the change: removing the duplicate Matter server URL.
Docstring Coverage Passed 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 Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed 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. (1 skipped: 1 unsupported.)

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

Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 274beceb-0d71-482b-b717-d4273d42badf
Commits

Reviewing files that changed from the base of the PR and between 169668c and 723b55b.

Files selected for processing: 1

  • matter_server/config.yaml

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

@sairon sairon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It was fixed in home-assistant/app-matter-server@6e3c0ac so another release of Matter server will propagate it here, but no harm if we do it in advance manually. Thanks!

@sairon
sairon merged commit 4d7840e into home-assistant:master Nov 11, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants