-
Notifications
You must be signed in to change notification settings - Fork 4
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
Diskussionen / Neuerungen / Hinweise #49
Comments
Grundsätzlich spricht nichts dagegen. Toll wäre, wenn wir es programmatisch gegen die Version eines Moduls auswerten könnten. |
Wie stellst du dir das vor @sidey79 Es wäre auf jedenfall falsch, wenn durch ein Fremdmodulfehler deine Tests fehl schlagen. Das würde nur Sinn machen bei eigenen Modulen. |
@HomeAutoUser
Hab es nicht näher durchdacht :) |
Was soll dann passieren, wenn die Bedingung (nicht) zutrifft? |
@elektron-bbs |
Dann werden im Laufe der Zeit sicher die meisten Tests übersprungen :-) |
Das einfache überspringen ist denke ich nicht Sinnvoll auf die Dauer. Solltest du dies so anstreben, so macht es nur Sinn, das übersprungene Tests registriert werden mit einem Hinweis. Daraus resultiert, wer oder was geht dem übersprungenem Ergebnis nach? Da würde man sich zusätzliche Arbeit verschaffen. Die Grundsatzfrage ist, wie weit geht man oder lehnt sich aus dem Fenster. Die Registrierung der Modulrevision und das Datum vom „Check / added / updated Eintrag“ verhilft uns um eine Bewertung und bessere Einsortierung der Timeline bei Fehlern. Die grundsätzliche Aufgabe liegt ja beim Maintainer es nicht zu verschlimmbessern :) Natürlich kannst du bei freier Zeit und Wille dir die Arbeit verschaffen um fremdes zu prüfen aber dazu gehört dann auch die Pflege der Daten / Weitergabe bzw. Bekanntgabe der Eventuell auftretenden Daten. Da fällt mir ein, was wertet man als Fehler? Da kann ggf auch nur eine Umbenennung einen Fehler verursachen was aber kein „richtiger Fehler ist“. |
@HomeAutoUser Du wolltest "revision_modul“ und „revision_entry" hinterlegen. Ich versuche mein Verständnis darzustellen: "revision_modul“ = Versionsnummer des logischen Modules, welches die Daten verarbeitet. Um das einfach mal grafisch zu Zeigen: "revision_entry“ = Das ist programmatisch vermutlich nicht auszuwerten. Wünschen würde ich dann aber noch eine 3. Information. "minProtocolVersion" = Die Versionsnummer von SD_ProtocolData ab dem die RMSG ausgewertet werden kann. |
@sidey79
Soll eher als Zusatzinformation dienen um zu sehen von wann dies stammt bzw. um bei update des Eintrages (weil vielleicht das Modul sich änderte) den Moment festzuhalten.
Deine Überlegung verstehe ich. Da würde mir als Umsetzung einfallen, das man nur bei „Ersteinträgen“ der Wert geschrieben wird. Alle Werte welche jetzt drin sind erhalten „unknown“. Da könnte man mit Fleiß in den Commits wühlen wann es dazu kam und manuell ergänzen. |
Die Arbeit, bei allen herauszufinden mit welcher Version das Protokoll aufgenommen wurde, würde ich auch niemandem zumuten. Anstelle von unknown würde ich eher die aktuelle hinterlegen. |
@sidey79
Ich würde es so einpflegen und bei jedem Eintrag der hinzu kommt, wird der Punkt |
minProtocolVersion wäre nur zu ändern, wenn es mit einer vorherigen Version nicht mehr funktioniert denke ich. Was ist der Hintergrund als revision mehrere Angaben (Dateiname, SVN Commit, Zeitstempel und Author) zu erfassen? |
Hintergrund die Daten revision_modul mit den Syntax zu füllen entspricht der Ausgabe von der Versionsausgabe vom Modul ;) Gib doch mal |
Für das o.g. habe ich das Thema erstellt.
Neuerung welche ins Pre-Release kommt demnächst. Die JSON möchte ich ergänzen mit „revision_modul“ und „revision_entry“.
revision_modul: speichert den Modulstand (Version) vom verarbeitenden Modul
revision_entry: speichert das Datum / Zeit wo der Eintrag geupdatet/erstellt wurde
Die Eintragungen bringen uns bei einer Bewertung oder Fehlersuche weiter. So wird die Sprungmarke ersichtlich wo es funktionierte.
@sidey79 @elektron-bbs spricht da von Euch was dagegen? Die Erläuterung erfolgt kurz und knapp weil vom Handy schreibe.
The text was updated successfully, but these errors were encountered: