-
Notifications
You must be signed in to change notification settings - Fork 1
Fix/installer tests readme #14
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
Leitfaden für ReviewerDieser PR verbessert die Dokumentationskonsistenz in der deutschen README-Datei und verfeinert das Installationsskript, indem ungenutzte Variablen entfernt, die Formatierung verbessert und die korrekte Handhabung von systemd-resolved sichergestellt wird. Änderungen auf Dateiebene
Tipps und BefehleInteraktion mit Sourcery
Ihr Erlebnis anpassenGreifen Sie auf Ihr Dashboard zu, um:
Hilfe erhalten
Original review guide in EnglishReviewer's GuideThis PR enhances documentation consistency in the German README and refines the installer script by removing unused variables, improving formatting, and ensuring proper systemd-resolved handling. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
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.
Hallo! Ich habe deine Änderungen überprüft – hier ist etwas Feedback:
- Der
apt-get install-Block in install.sh hat seinapt-get install -y-Präfix nach dem Zeilenumbruch verloren und wird die benötigten Pakete stillschweigend nicht installieren. - Du hast die Standard-SUITE_API_KEY-Generierung entfernt, wodurch der Installer eine extern gesetzte Umgebungsvariable erfordert – führe entweder den Fallback wieder ein oder aktualisiere die README, um darauf hinzuweisen.
- Überprüfe noch einmal, ob die neue Funktion
handle_systemd_resolvedfrüher im Skript definiert ist und das Deaktivieren von systemd-resolved korrekt handhabt, bevor Pi-hole startet.
Prompt für KI-Agenten
Bitte beachte die Kommentare aus dieser Code-Überprüfung:
## Allgemeine Kommentare
- Der `apt-get install`-Block in install.sh hat sein `apt-get install -y`-Präfix nach dem Zeilenumbruch verloren und wird die benötigten Pakete stillschweigend nicht installieren.
- Du hast die Standard-SUITE_API_KEY-Generierung entfernt, wodurch der Installer eine extern gesetzte Umgebungsvariable erfordert – führe entweder den Fallback wieder ein oder aktualisiere die README, um darauf hinzuweisen.
- Überprüfe noch einmal, ob die neue Funktion `handle_systemd_resolved` früher im Skript definiert ist und das Deaktivieren von systemd-resolved korrekt handhabt, bevor Pi-hole startet.Sourcery ist kostenlos für Open Source – wenn dir unsere Überprüfungen gefallen, ziehe bitte in Betracht, sie zu teilen ✨
Original comment in English
Hey there - I've reviewed your changes - here's some feedback:
- The apt-get install block in install.sh lost its
apt-get install -yprefix after the line break and will silently fail to install the needed packages. - You removed the default SUITE_API_KEY generation, making the installer require an externally set env var—either reintroduce the fallback or update the README to call this out.
- Double-check that the new
handle_systemd_resolvedfunction is defined earlier in the script and correctly handles disabling systemd-resolved before Pi-hole starts.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The apt-get install block in install.sh lost its `apt-get install -y` prefix after the line break and will silently fail to install the needed packages.
- You removed the default SUITE_API_KEY generation, making the installer require an externally set env var—either reintroduce the fallback or update the README to call this out.
- Double-check that the new `handle_systemd_resolved` function is defined earlier in the script and correctly handles disabling systemd-resolved before Pi-hole starts.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Zusammenfassung von Sourcery
Restrukturiert und bereinigt das gesamte Projekt durch Refactoring des Ein-Klick-Installers und der Dienststartlogik, Modernisierung und Vereinfachung des API- und Schema-Codes, Überarbeitung der Dokumentation und Straffung der Tests bei gleichzeitiger Entfernung von totem Code.
Verbesserungen:
Dokumentation:
Tests:
Original summary in English
Zusammenfassung von Sourcery
Überarbeitung des deutschen README mit konsistenter Markdown-Formatierung, Beispielen und aktualisierter Befehlsnutzung, sowie Optimierung von install.sh durch Entfernen ungenutzter Variablen, Verbesserung der Formatierung, Hinzufügen der systemd-resolved-Handhabung und eines Hinweises zur DoT-Weiterleitung.
Verbesserungen:
Dokumentation:
/leases-Endpunkt und Aktualisierung der Nutzung des API-Schlüssel-Headers in der API-ReferenzWartungsarbeiten:
Original summary in English
Summary by Sourcery
Refine the German README with consistent markdown formatting, examples, and updated command usage, and streamline install.sh by removing unused variables, polishing formatting, and adding systemd-resolved handling and a DoT forwarding note.
Enhancements:
Documentation:
Chores: