Tags in der Doku

[betateilchen]

wenn man ihm eine hand voll feste regeln vorgibt?

Weil sich in fhem schon jetzt kaum jemand (nichtmal altgediente Entwickler!) an bereits seit langem geltende, simpelste Regeln hält. Bestes Beispiel: Die von Rudi festgelegte 80-Zeichen Begrenzung in CHANGED.

[justme1968]

weil man diese festen regeln dann im commit überprüfen kann. das wäre für die 80 zeichen übrigens auch besser als sich jedes mal drüber zu ärgern.

[betateilchen]

es steht Dir frei, einen entsprechenden pre-commit hook bereitzustellen, den Rudi dann sicher gerne einbaut.

[Markus Bloch]

Anbei ein Patch um die 80 chars/line Regel auf gewünschte Files zu prüfen (aktuell nur CHANGED). Allerdings ungetestet.

Viele Grüße

Markus

[rudolfkoenig]

@Markus: Danke, habs eingecheckt, aktiviert, gefixt, dann einige uralte CHANGED Zeilen gefixt, und dann nochmal eingecheckt.

[hexenmeister]

könnt mich hauen, aber ich sehe keinen Fehler...
In dieser Tabelle (war im SYSMON-Doku) sollen table und td 'unbalanced' sein.

Code Auswählen Erweitern

EN FHEM/42_SYSMON.pm: Unbalanced table (-1, last line ok: 4693)
EN FHEM/42_SYSMON.pm: Unbalanced td (-66, last line ok: 4443)

Aber wo?? Nach dem Entfernen zweier solchen Tabellen (DE und EN) konnte ich einchecken...
Bin ich blind, oder hat die Prüfmmethode einen Fehler?

Code Auswählen Erweitern

      <table style="border: 1px solid black;">
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>cpu_freq</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>900</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>cpu_temp</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>49.77</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>cpu_temp_avg</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>49.7</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>eth0</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>RX: 2954.22 MB, TX: 3469.21 MB, Total: 6423.43 MB</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>eth0_diff</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>RX: 6.50 MB, TX: 0.23 MB, Total: 6.73 MB</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>fhemuptime</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>11231</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>fhemuptime_text&nbsp;&nbsp;</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>0 days, 03 hours, 07 minutes</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>idletime</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>931024 88.35 %</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>idletime_text</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>10 days, 18 hours, 37 minutes (88.35 %)</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>loadavg</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>0.14 0.18 0.22</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>ram</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>Total: 485 MB, Used: 140 MB, 28.87 %, Free: 345 MB</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>swap</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>n/a</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>uptime</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>1053739</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>uptime_text</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>12 days, 04 hours, 42 minutes</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>wlan0</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>RX: 0.00 MB, TX: 0.00 MB, Total: 0 MB</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>wlan0_diff</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>RX: 0.00 MB, TX: 0.00 MB, Total: 0.00 MB</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>fs_root</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>Total: 7404 MB, Used: 3533 MB, 50 %, Available: 3545 MB at /</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>fs_boot</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>Total: 56 MB, Used: 19 MB, 33 %, Available: 38 MB at /boot</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>fs_usb1</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>Total: 30942 MB, Used: 6191 MB, 21 %, Available: 24752 MB at /media/usb1&nbsp;&nbsp;</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>stat_cpu</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>10145283 0 2187286 90586051 542691 69393 400342&nbsp;&nbsp;</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>stat_cpu_diff</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2151 0 1239 2522 10 3 761&nbsp;&nbsp;</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td style="border-bottom: 1px solid black;">
               <div>stat_cpu_percent</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>4.82 0.00 1.81 93.11 0.05 0.00 0.20&nbsp;&nbsp;</div>
            </td>
            <td style="border-bottom: 1px solid black;">
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
         <tr>
            <td>
               <div>stat_cpu_text</div>
            </td>
            <td>
               <div>user: 32.17 %, nice: 0.00 %, sys: 18.53 %, idle: 37.72 %, io: 0.15 %, irq: 0.04 %, sirq: 11.38 %&nbsp;&nbsp;</div>
            </td>
            <td>
               <div>2013-11-27 00:05:36</div>
            </td>
         </tr>
      </table>

[Markus Bloch]

siehe Zitat:

DAS ist mir schon klar, es geht in dieser Diskussion gerade darum, dass ich keine Attribute sehen will.

Sonst fangen die Leute mit "style='color:light-blue;font-size:48px'" und Vergleichbares an. Das habe ich fuer ul vor (gefuehlt) 2 Jahren einmal schon unterbunden, jetzt folgte nur table & co, da ich damals nicht auf die Idee kam, das man table in der Doku verwenden koennte.

<td>, <table>, <tr> dürfen keine Attribute haben.

[hexenmeister]

Aha!  Danke!

[Sailor]

Moin zusammen

<td>, <table>, <tr> dürfen keine Attribute haben.

Aha! Danke!

Gruss
    Sailor

[Dr. Boris Neubert]

Hallo,

irgendwie war doch hier die vorherrschende Meinung, dass wir in der commandref Attribute in Tags erlauben sollen. Dann kam aber keine Aktion mehr dazu.

Die Unbalanced-Meldungen können m.E. so nicht bleiben, zumal sie dem Anwender bei jedem Update angezeigt werden.

Viele Grüße
Boris

[betateilchen]

Die Unbalanced-Meldungen können m.E. so nicht bleiben, zumal sie dem Anwender bei jedem Update angezeigt werden.

Spätestens beim nächsten Update eines betroffenen Moduls löst sich das von selbst, da es solange nicht eingecheckt werden kann, solange es unbalanced Meldungen gibt. Und das finde ich auch gut so 

Dass "ja zu Attributen" eine vorherrschende Meinung gewesen sein soll, kann ich übrigens nicht nachvollziehen.

[Dr. Boris Neubert]

Habe ich das was überlesen? Es gab hier mehrere Stimmen, die sich für Attribute ausgeprochen haben. Gibt es jetzt einen precommit check?

Das einzige Gegenargument, das ich gesehen habe, war die Übersetzung von HTML nach Plaintext. Und dazu fällt es mir schwer zu glauben, dass es sich dabei um ein Problem handelt, dass Du nicht lösen könntest.

Viele Grüße
Boris

[herrmannj]

bin auch dafür.

Schlussendlich aber Rudis Kompetenzbereich, und der hat nö gesagt. Muss man akzeptieren, geht ja auch so.

vg
joerg

[betateilchen]

Habe ich das was überlesen? ... Gibt es jetzt einen precommit check?

Ja, offenbar hast Du da was überlesen.
Ja, es gibt jetzt einen pre-commit hook.

Schlussendlich aber Rudis Kompetenzbereich, und der hat nö gesagt.

*gefällt mir*

[Dr. Boris Neubert]

Ja, offenbar hast Du da was überlesen.
Ja, es gibt jetzt einen pre-commit hook.

Bitte zeige mir den Beitrag. Ich habe Tomaten auf den Augen.