commandref_join.pl Warnungen

Begonnen von RichardCZ, 18 April 2020, 21:38:06

Vorheriges Thema - Nächstes Thema

RichardCZ

Jedesmal wenn ich bei mir einen "Make update" mache, kommen beim Lauf von commandref_join.pl  diese Warnungen:

*** EN SISPM line 570: div with attributes (apart from class) is not allowed
*** EN FHEM/70_SISPM.pm: Unbalanced div (1, last line ok: 569)
*** EN WifiLight line 4892: table with attributes (apart from class) is not allowed
*** EN FHEM/32_WifiLight.pm: Unbalanced table (1, last line ok: 4891)
*** DE WifiLight line 5537: table with attributes (apart from class) is not allowed
*** DE FHEM/32_WifiLight.pm: Unbalanced table (1, last line ok: 5536)


ist das nur bei mir oder sind die Pappenheimer bekannt?
Witty House Infrastructure Processor (WHIP) is a modern and
comprehensive full-stack smart home framework for the 21st century.

CoolTux

Das ist bei allen. Zu mindest bei allen den die Warnungen nicht ausschalten. Bei einem normalen Update beim User wird das generieren mit nowarn aufgerufen.
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

Wzut

gibt es vllt. auch die Möglichkeit diese Prüfung auch svn seitig bei einem commit laufen zu lassen und ggf. wie bei anderen Fehlern dann einfach abzulehnen ?
Maintainer der Module: MAX, MPD, UbiquitiMP, UbiquitiOut, SIP, BEOK, readingsWatcher

CoolTux

Ich weiß es nicht, bin aber der Meinung daß dies seit kurzem so gemacht wird. Bin mir aber echt unsicher.
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

rudolfkoenig

Diese Pruefung habe ich am 2019-12-06 eingebaut, sie ist auch Teil der SVN pre-commit, die beiden Dateien wurden aber seitdem nicht angefasst.
Die Diskussion dazu ist hier dokumentiert.
Das ironische an der Sache ist, dass die drei angemeckerten Probleme eigentlich keine sind, da die "Probleme" auskommentiert sind.
Da ich aber im Hook nicht mit HTML-Kommentarerkennung anfangen will, habe ich die beiden Module geaendert.

herrmannj

Hat also richtig viel gebracht der thread :) Danke @Rudi