-
Notifications
You must be signed in to change notification settings - Fork 15
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
Think about to fix the issues found by adapter checker #15
Comments
[E130] No news found in io-package.json in fc34255 |
Ok hab jetzt alles gemacht, was ich machen kann. Und bei Travis anmelden und die Testruns aktivieren. Anmelden mit github account und dann freigabe aufs repository einrichten. Dann müsste der Test laufen. Falls da was nicht geht, frag am besten @Apollon77, der hilft einem da immer weiter. |
Topic ist im prinip ein link wo der Adapter im Forum getetstet wurde? Und bei Travis anmelden und die Testruns aktivieren. hatte ich schonmal gemacht sonst muss ich das nochmal tun |
ich lasse gerade alles durch Appveyor und Travis durchlaufen... |
Ok, in travis und appveyor bitte node 4 und 6 wegwerfen (js-controller ist nur ab nodejs 8 ...) und dafür 10 und 12 hinzufügen |
wird gemacht. |
Und naja ansonsten fehlt das "test" Verzeichnis :-) Damit kann testing nicht tun ... DU kannst (wenn DU nicht alles neu auf Basis des Creators aufbauen willst) ggf unter https://github.com/ioBroker/ioBroker.template/tree/master/JavaScript anleihen nehmen und Teile kopieren |
ok, hab ich auch erledigt. |
Na klick doch mal rein ... :-) DU musst dann auch das testing modul in den devDependencies mit rein nehmen und ggf noch paar mehr. |
ich habe jetzt zu geschrieben.... aber ob das alles richtig ist weiß ich nicht.... |
|
Adapter etsting ist ok, die Fehler bedeuten:
|
ok punkt 1 und 2 hab ich schon erledigt. |
punkt 4 hab ich hoffentlich richtig gemacht .... 8b269f0 |
Ja ist korrekt, dann hast Du aber eine Abhängigkeit zum controller 1.5.14 (unter dem geht es nicht). adapter.terminate ? adapter.terminate(1) : process.exit(1); |
also müsste ich es so schreiben? |
beispielsweise |
Sorry das ich immer so doof nachfrage, ist leider nicht mein tagesgeschäft so ein adapter zu programieren... JAVA usw. Ich hoffe nur ich stelle mich nicht zu blöde an... |
sollte so tun ... und wir helfen immer gern |
Ja das hab ich schon oft bemerkt, bin auch sehr dankbar dafür 👍 |
Kann es denn jetzt ins repository? |
Irgendwie ist travis aber immer noch rot und dinge offen die du an sich gesagt hast gefixt zu haben ?! https://travis-ci.org/Matten-Matten/ioBroker.enigma2/jobs/623124650#L706 |
Ich glaube ich weiß warum, ich muss das bestimmt erst alles nach npm Portieren oder? |
ne travis läuft mit der github version |
Ok, mal ein bissl Review-Kommentare:
|
punkt 1 keine ahnung was du damit meinst.... sry. |
Hey, nicht den Kopf hängen lassen. Sieht doch gut aus der Adapter. Das mit den Tests kann etwas fummelig sein, da hatte ich auch ein bisschen zu kämpfen. 🔨 Darf ich noch ein paar Kommentare geben, die vielleicht bei der Klärung helfen?
adapter.on("unload", function () {
if (pollIntervalId1) {
clearInterval(pollIntervalId1);
}
...
} (Bei ... müsste dann das gleiche nochmal für die anderen beiden setInterval-ids gemacht werden, falls du die beibehälst, und natürlich den Variablennamen anpassen). Ich hoffe, das war jetzt nicht zu übergriffig. Ich finde du hast dir doch schon viel Arbeit gemacht und aus meiner Sicht fehlt doch gar nicht mehr viel. Daher fände ich es schön, wenn du jetzt die Kleinigkeiten auch noch gefixt bekommst und würde gerne dabei helfen. 😃 |
@Matten-Matten - Ich finde es auch toll dass du dich da engagierst. Es ist noch kein Meister vom Himmel gefallen und hey... scheinbar macht dir sowas Spaß, sonst hättest du es gar nicht angefangen. 😃 Klar.... die Hürden damit was aufgenommen wird sind schon hoch, was manchmal nervig ist, aber es hat alles ein Sinn. Wir wollen ioBroker stabil halten und da muss Kontrolliert werden. Aber dafür haben wir die Entwickler-Community: https://github.com/iobroker-community-adapters/info - hier bekommt man immer Unterstützung. Schau doch mal rein! Wir freuen uns immer über Zuwachs... :-) |
OK, OK, OK,..... @Garfonso
ist das so richtig? |
wenn ich den adapter mit den änderungen stoppe kommt volgendes als Warnmeldung: `
` was ich nicht verstehe ist er sagt ja event_interval is not defined aber er ist doch mit var definiert.... oder liegt das daran das er in einer funktion definiert ist. muss ich ihn jetzt außerhalb der funktion definieren? |
EDIThabs jetzt so erstellt:
jetzt steht das im LOG wenn ich den Adapter stoppe: `
` |
wenn das so ok sein sollte würde ich das im Adapter übertragen. EDIThab alles übertragen aber die Tests fallen trotzdem durch: |
Erstmal: Genau so meinte ich das (also zweite Variante). Die Variablen müssen außerhalb der Funktion sein, sonst sind sie in der anderen Funktion nicht bekannt (der Raum in dem eine Variable gilt nennt sich "scope", falls du dich weiter damit beschäftigen willst). Hm.. wo kommt denn diese 10 her?
|
Do you need help fixing the bugs? |
@Matten-Matten remove the unit test from travis and you shoud be good |
@Apollon77 |
lösch einfach unit.js aus https://github.com/Matten-Matten/ioBroker.enigma2/tree/master/test ... ist zwar nicht sauber aber ok-ish |
ok. Danke. |
- (mcm1957) The adapter has been moved into iobroker-community-adapters organisation - (mcm1957) changed: Testing has been added and changed to support node 16, 18 and 20 - (mcm1957) changed: Dependencies have been updated - (mcm1957) changed: issues reported by adapter-checker have been fixed (#15)
fixed 1.5.0 |
Thanks, that all bugs have been fixed. |
I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/Matten-Matten/ioBroker.enigma2
I have also found warnings that may be fixed if possible.
Thanks,
your automatic adapter checker.
P.S.: There is a community in Github, which supports the maintenance and further development of adapters. There you will find many experienced developers who are always ready to assist anyone. New developers are always welcome there. For more informations visit: https://github.com/iobroker-community-adapters/info
The text was updated successfully, but these errors were encountered: