-
Notifications
You must be signed in to change notification settings - Fork 0
Feat/dependabot daily updates #38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Add migration guide: homeassistant/matter-migration.md - Document migration from t0bst4r/home-assistant-matter-hub to Luligu/matterbridge - Add EOL warning to README.md for Matter Hub service - Upstream project end-of-maintenance since January 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This pull request updates the Dependabot configuration to check for updates daily instead of weekly, and adds comprehensive Matter Hub migration documentation.
Changes:
- Changed Dependabot Docker Compose update interval from weekly (Sunday) to daily for all service directories (homeassistant, paperless, immich, pihole, freshrss, teslamate)
- Added a Matter Hub deprecation notice in README.md with a link to migration documentation
- Added a new 280-line migration guide (in German) for migrating from home-assistant-matter-hub to Matterbridge
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
.github/dependabot.yml |
Updated all service update schedules from weekly to daily intervals |
README.md |
Added deprecation warning for Matter Hub with link to migration documentation |
homeassistant/matter-migration.md |
New comprehensive migration guide from matter-hub to Matterbridge (contains critical YAML syntax errors) |
Comments suppressed due to low confidence (3)
homeassistant/matter-migration.md:133
- Inconsistent directory naming: Line 122 references
./Matterbridge(capital M) while line 133 createsmatterbridge(lowercase m). Additionally, line 124 references./mattercertbut this directory is not created in line 133. Either update line 133 to create all three directories with matching case (Matterbridge,matterbridge-data,mattercert), or update lines 122 and 124 to match the lowercase convention.
docker compose up -d matterbridge
homeassistant/matter-migration.md:1
- The PR title "Feat/dependabot daily updates" suggests the primary focus is updating Dependabot configuration, but this PR also includes a complete 280-line Matter migration documentation file (
homeassistant/matter-migration.md). Consider splitting this into two separate PRs: one for the Dependabot configuration change and another for the Matter migration documentation. This would make the changes easier to review and maintain a clearer git history.
# Migration: home-assistant-matter-hub → Matterbridge
homeassistant/matter-migration.md:117
- The Docker image reference for Matterbridge uses the mutable
latesttag (image: luligu/matterbridge:latest), which creates a supply-chain risk because future image updates (or a compromised registry/publisher) can silently change the code you run with access to your Home Assistant token and host network. An attacker who gains control over that tag could ship a malicious image that is automatically pulled and executed with your configured volumes and privileges, leading to compromise of your home automation environment. Pin this dependency to an immutable identifier (such as a specific image digest or at least a concrete, audited version) instead oflatestto ensure only trusted builds are deployed.
image: luligu/matterbridge:latest
container_name: matterbridge
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| schedule: | ||
| interval: "weekly" | ||
| day: "sunday" | ||
| interval: "daily" |
Copilot
AI
Feb 3, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changing Dependabot update frequency from weekly to daily will significantly increase the number of pull requests (potentially 7x more PRs per week). This could create maintenance overhead and noise in the repository. Consider whether daily updates are truly necessary for all services, or if a weekly schedule is sufficient for most while keeping daily updates only for internet-facing services like the proxy.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot Ja, das ist richtig. Aber auf der anderen Seite enthalten die Versionen immer auch Bug und Security Fixes, die ich gerne Zeitnahe einspielen möchte.
| @@ -118,11 +118,6 @@ Ersetze in `docker-compose.yml` den `matter-hub` Service: | |||
| restart: unless-stopped | |||
| network_mode: host | |||
| volumes: | |||
Copilot
AI
Feb 3, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The volumes: key is duplicated on lines 120 and 121. This will cause a YAML syntax error when used in docker-compose. Remove the first occurrence on line 120.
| volumes: |
No description provided.