extension that the script also uses the local data for checking #323

oweitman · 2024-10-28T00:11:10Z

wie auf discord schon geschrieben, hier die Erweiterung,
so das rein lokal geprüft werden kann.
ich habe das mit allen meinen Adaptern getestet, ob da alle Eventualitäten abgedeckt sind kann ich nicht sagen.

Die Anpassung enthält grob die folgenden Anpassungen

Prüfung ob zusätzlicher Parameter --local in der Kommandozeile enthalten ist
common.js
Anpassung der Funktion downloadFile, so das bei isLocal die Datei aus dem Dateisystem geladen wird, anstatt von github
M100IOPackage: Bei der Überprüfung des icons wird die url aus der io-package.json genommen. hier war eine extra Behandlung zur Extrahierung des Dateinamens aus der url notwendig
M500_Code: Die Prüfung der konkreten Dateien erfolgt ja aus einer Zip-Datei, die von Github geladen wird. Um die erforderliche Datenstruktur für die Folgeverarbeitung zu erhalten, musste hier ein Suchalgorithmus für die locale Dateien eingefügt werden (Funktion getAllFiles). Das Array skipDirectories könnte man ggfs aus .gitignore nehmen, aber so war es erst einmal wesentlich einfacher. Das dürfte aber auch keine Probleme geben. ggfs. müsste die Liste erweitert werden, falls noch Fälle auftreten, die mir aktuell nicht bekannt sind.

Wie das Skript aktuell getestet werden kann, steht in der Datei Debugging.md. Falls der PR nach ausführlichem Test in das Repo mit aufgenommen wird, kann diese wieder entfernt werden.

Ich habe auch mal über alle Dateien eslint und prettier laufen lassen.
Daher sind noch relativ viele Änderungen auch in den anderen Dateien, die aber meist nur aus einfügen/entfernen von Leerzeichen und Einrückungen besteht.

…pository

- (oweitman) extension that the script also uses the local data for checking.

lib/M000_PackageJson.js

            context.errors.push(`[E049] "keywords" within package.json must contain some keywords besides "${context.packageJson.keywords.join(', ')}" related to adapter`);
        }
-        if (! recommendedKeywords.filter(keyword => context.packageJson.keywords.includes(keyword)).length > 0) {
+        if (!recommendedKeywords.filter(keyword => context.packageJson.keywords.includes(keyword)).length > 0) {


scripts/updateAllIssues.js

-            if(!check){
-                check = await checkIoPackage("https://raw.githubusercontent.com/" + full_name + "/main/io-package.json", full_name);
+            console.log('Check ' + full_name);
+            let check = await checkIoPackage('https://raw.githubusercontent.com/' + full_name + '/master/io-package.json', full_name);


scripts/updateAllIssues.js

+            console.log('Check ' + full_name);
+            let check = await checkIoPackage('https://raw.githubusercontent.com/' + full_name + '/master/io-package.json', full_name);
+            if (!check) {
+                check = await checkIoPackage('https://raw.githubusercontent.com/' + full_name + '/main/io-package.json', full_name);


scripts/updateAllIssues.js

-                    let check = await checkIoPackage("https://raw.githubusercontent.com/" + full_name + "/master/io-package.json", full_name);
-                    if(!check){
-                        check = await checkIoPackage("https://raw.githubusercontent.com/" + full_name + "/main/io-package.json", full_name);
+                    let check = await checkIoPackage('https://raw.githubusercontent.com/' + full_name + '/master/io-package.json', full_name);


scripts/updateAllIssues.js

-                        check = await checkIoPackage("https://raw.githubusercontent.com/" + full_name + "/main/io-package.json", full_name);
+                    let check = await checkIoPackage('https://raw.githubusercontent.com/' + full_name + '/master/io-package.json', full_name);
+                    if (!check) {
+                        check = await checkIoPackage('https://raw.githubusercontent.com/' + full_name + '/main/io-package.json', full_name);


mcm1957 · 2024-10-28T07:30:02Z

DANKE für die Erweiterungen.
Werd mir das ansehen sobald ich Zeit habe.

oweitman · 2024-10-29T19:13:21Z

👍

oweitman added 7 commits October 26, 2024 19:59

adding the ability to run the script locally and testing the local re…

73cfcdb

…pository

fix some md issues

6656034

some clearifications

03036e0

Merge branchchanges from upstream

f0ff466

add documentation for start without installing

db8714e

prepare for release

67d12dc

chore: release v3.2.0

4921ebe

- (oweitman) extension that the script also uses the local data for checking.

github-advanced-security bot found potential problems Oct 28, 2024

View reviewed changes

add --debug as a parameter for conditional create extra output

cc7c124

mcm1957 merged commit eb5cd98 into ioBroker:master Oct 29, 2024
1 check passed

mcm1957 added a commit that referenced this pull request Oct 29, 2024

Revert changes of PR #323 related to scripts dir

ce747b3

mcm1957 added a commit that referenced this pull request Oct 29, 2024

Revert changes of PR #323 related to package.json

2925920

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

extension that the script also uses the local data for checking #323

extension that the script also uses the local data for checking #323

oweitman commented Oct 28, 2024 •

edited

Loading

mcm1957 commented Oct 28, 2024 •

edited

Loading

oweitman commented Oct 29, 2024

extension that the script also uses the local data for checking #323

extension that the script also uses the local data for checking #323

Conversation

oweitman commented Oct 28, 2024 • edited Loading

mcm1957 commented Oct 28, 2024 • edited Loading

oweitman commented Oct 29, 2024

oweitman commented Oct 28, 2024 •

edited

Loading

mcm1957 commented Oct 28, 2024 •

edited

Loading