Kann ich dich bitten, kurz einen Blick auf den neuen PR zu werfen: [CANCELED #2014] - Netatmo add valves NRV by Terdious · Pull Request #2005 · GladysAssistant/Gladys · GitHub, um zu bestätigen, dass alles in Ordnung ist?
Ich möchte sicher sein, dass die Methodik stimmt, wenn man einen PR von einem anderen Branch aus erstellt und welche Auswirkungen die Commit-Historie hat, wobei in diesem neuen PR die Historie des anderen PRs zu sehen ist. Und dass bei der Erstellung des neuen PRs die Tests des vorherigen erneut gestartet wurden… immer ein bisschen beunruhigend, wenn man das nicht oft macht ^^
Hallo @pierre-gilles!
Ja also, wenn man es so macht wie ich, ist das normal, ich habe bisher nichts gepusht, ich habe lokal gearbeitet.
Genau deshalb meine Frage. Ich bin von meinem Basis-Branch netatmo-integration ausgegangen, um meinen neuen Branch basierend auf origin/gladys zu erstellen (um mit dem Code dieses ersten PRs zu beginnen und mit der Integration fortzufahren). Ich habe nur den neuen Branch gepusht und dann den neuen PR erstellt, der mit dem GladysAssistant-Repo verknüpft ist.
Ist das die richtige Methode, um sauber zu bleiben?
Ich werde mit den Modulen der Wetterstation eines nach dem anderen weitermachen.
in der Hoffnung, dass die erste PR gut genug ist ^^ aber normalerweise habe ich die Zweige angehängt, damit die Aktualisierungen vereinfacht werden. Das sollte die anderen PRs nicht zu sehr beeinflussen, falls Änderungen in der Überprüfung verlangt werden!!
Edit:
@PhilippeMA, ich informiere dich, wenn ich mit der vollständigen Station fertig bin, damit du testen kannst, wenn es immer noch in Ordnung ist!
Danke für das Feedback! Für den Wettbewerb würde ich empfehlen, bei maximal 2 zu bleiben, für Dinge, die so im Hintergrund laufen. Es gibt Benutzer auf Pi, wir wollen eine leichte Software bleiben und große Spitzen vermeiden
Halte mich auf dem Laufenden, sobald die Dokumentation aktualisiert ist. Teste alles noch einmal gründlich, um sicherzustellen, dass du nichts kaputt gemacht hast, als du die Korrekturen vorgenommen hast, und für mich können wir dann die erste Version in die Produktion bringen
Neue vollständige Tests auf neuer Installation nach den Fixes der Überprüfung:
Dokumentation aktualisiert mit den aktuellen Bildern (en und fr):
Hinzufügen einer Dashboard-Ansicht zum Abschluss:
Docker-Image terdious/gladys:netatmo-integration-dev bei Bedarf aktualisiert:
Edit: Die folgenden 4 PRs sind zur Überprüfung bereit mit den Aktualisierungen der ersten. Du kannst dich damit beschäftigen, sobald du Zeit hast, aber nur zur Info, sie sind wirklich sehr leicht, wie du sehen wirst. Bisher habe ich nur das hinzugefügt, was bereits existiert (außer bei der ersten der Wetterstation, die 1 oder 2 zusätzliche Konstanten erforderte, aber nichts Schlimmes).
Docker-Image terdious/gladys:netatmo-add-module-outdoor-weather-station für diejenigen, die es wünschen:
Regenmesser (einfach) und Anemometer (etwas schwieriger - Front-Dashboard-Hinzufügung) in Arbeit:
Edit: Es ist hässlich, aber mein einziger Weg, sie erneut auszuführen, ist ein doppelter „Fake Push“…
Bei mir tritt das Problem eindeutig auf, sobald mindestens 2 PRs in den Tests sind. Gestern ist das dreimal passiert, als ich in kurzer Zeit Pushes auf verschiedene PRs gemacht habe.
Letztendlich bin ich etwas zurückgegangen, habe die Zweige und PRs vom aktuellen Master aus neu erstellt. Es war zu umständlich, sie zu aktualisieren, und jetzt ist es viel sauberer.
Ich habe also 2 PRs zur Überprüfung bereit:
@pierre-gilles, kannst du die 6 PRs überprüfen (ich habe sie unabhängig gemacht, aber die Reihenfolge ist zumindest für PR 1 und 2 wichtig).
Und du sagst mir, ob du es vorziehst, ich habe einen Zweig, der alle 6 Geräte integriert. Das kann die kleinen Korrekturen, die nach und nach vorgenommen werden, erleichtern und vor allem die Überprüfung (46 Dateien im einzigen Zweig gegen 6 PRs mit jeweils etwa 20 Dateien, da die meisten Dateien in jeder PR betroffen sind).
@PhilippeMA, wenn du motiviert bist, die gesamte Wetterstation zu testen: docker pull terdious/gladys:netatmo-features-nrv-weather
Vergiss nur nicht, in den richtigen Branch zu committen.
Es ist kein Problem, wenn es schmutzige Commits gibt, da es nur in der PR ist, ich werde danach squash mergen.
