FHEM Forum

FHEM - Entwicklung => FHEM Development => Thema gestartet von: Markus Bloch am 20 August 2016, 22:51:18

Titel: Hinweis an alle Modulentwickler: neue Prüfung beim einchecken
Beitrag von: Markus Bloch am 20 August 2016, 22:51:18
Hallo zusammen,

ich möchte euch alle darauf aufmerksam machen, dass mit Revision 12018 ein neuer pre-commit Hook im SVN installiert wurde. Dadurch werden einige (oder sogar viele) von euch Fehlermeldungen beim Einchecken bekommen.

Hintergrund ist eine neue Commandref, welche im Rahmen von https://forum.fhem.de/index.php/topic,39854.msg482947.html#msg482947 implementiert wurde. Die neue modulare Commandref besitzt als Inhaltsverzeichnis eine Modulübersicht in Form einer Tabelle. Dort ist zusammen mit einem Link zur Commandref auch eine Kurzbeschreibung für jedes Modul aufgelistet.

Diese Kurzbeschreibung muss mind. in Englisch in jedem Modul vorhanden sein, damit dieses in SVN eingecheckt werden kann.

Da dies aktuell natürlich nicht überall der Fall ist, wird ein Commit nun verweigert, wenn keine Kurzbeschreibung im Modul vorhanden ist.

Um eine Kurzbeschreibung in eure Module einzufügen, müssen am Anfang der Commandref ein POD-Marker "=item summary" gesetzt werden:
Zitat=pod
=item device
=item summary    controls LG SmartTV's build between 2012-2014 via LAN connection
=item summary_DE steuert LG SmartTV's, welche zwischen 2012-2014 hergestellt wurden, via LAN-Verbindung

=begin html
...


=pod
=item command
=item summary    shows the version of FHEM and all loaded modules.
=item summary_DE zeigt die Version von FHEM und allen geladenen Modulen an

=begin html
....

Die Kurzbeschreibung ist auf eine Zeile begrenzt und darf maximal 80 Zeichen als reinen Beschreibungstext enthalten. Es geht hierbei darum ein grobes Verständnis zu vermitteln, wofür das jeweilige Modul gedacht ist. Bei Gerätemodulen ist dies in der Regel: "welche(s) Gerät(e) von welchem Hersteller wird über welche Verbindungsart gesteuert". Bei Hilfsmodulen/Befehlsmodulen sollte entsprechend beantwortet werden, was dieses Modul grob leistet/bereitstellt. Der Modulname muss dabei in der Beschreibung nicht enthalten sein, da der Modulname direkt vor der Kurzbeschreibung als Link steht.

Bitte daher mindestens eine englische Kurzbeschreibung via "=item summary" einfügen, damit ihr eure Module wieder einchecken könnt.

Viele Grüße

Markus


Titel: Antw:Hinweis an alle Modulentwickler: neue Prüfung beim einchecken
Beitrag von: Markus M. am 20 August 2016, 23:16:35
Danke für den Hinweis und die Beschreibung!
Für Allergy und Netatmo erledigt.
Titel: Antw:Hinweis an alle Modulentwickler: neue Prüfung beim einchecken
Beitrag von: Sidey am 14 Dezember 2019, 23:20:37
Hi Rudi,

ich glaube ich habe eine kleine Schwäche im precommit im Zusammenhang mit den 80 Zeichen gefunden.
Bei den 80 Zeichen, geht es doch darum, dass die Darstellung nicht gesprengt wird nehme ich an.

Folgendes funktioniert nicht, da in der Zeile mehr als 80 Zeichen stehen:

Unterstützt den gleichnamigen Low-Cost Empfänger für digitale Signale

Wenn ich die Umlaute durch ue / ae ersetze sind in der Zeile weniger als 80 Zeichen und es klappt dann:
Unterstuetzt den gleichnamigen Low-Cost Empfaenger fuer digitale Signale

Wäre es möglich die Umlaute nur als ein Zeichen zu berücksichtigen?

Grüße Sidey
Titel: Antw:Hinweis an alle Modulentwickler: neue Prüfung beim einchecken
Beitrag von: betateilchen am 15 Dezember 2019, 12:58:50
Zitat von: Sidey am 14 Dezember 2019, 23:20:37
Wäre es möglich die Umlaute nur als ein Zeichen zu berücksichtigen?

ich wäre eher für das bereits getestete Ersetzen durch ae, oe usw.