ioBroker / ioBroker.iot

Connection for Alexa, IFTTT, Google Home and co
MIT License
72 stars 25 forks source link

Please fix issues reported by adapter checkern and review testing #774

Open mcm1957 opened 3 months ago

mcm1957 commented 3 months ago

❗ [E034] No devDependency declared for @iobroker/testing. Please add "@iobroker/testing":"4.1.3" to devDependencies at package.json 👀 [W028] Minimum node.js version 18 recommended. Please adapt "{'engines' : { 'node' >= '16' } }" at package.json. 👀 [W105] Missing suggested translation into uk of "common.titleLang" in io-package.json. 👀 [W109] Missing suggested translation into uk of "common.desc" in io-package.json. 👀 [W156] Adapter should support admin 5 UI (jsonConfig) if you do not use a React based UI 👀 [W174] Potential sensitive data "pass" not listed at "encryptedNative" in io-package.json 👀 [W184] "common.materialize" is deprecated for admin >= 5 at io-package.json. Please use property "adminUI". 👀 [W184] "common.title" is deprecated and replaced by "common.titleLang". Please remove from io-package.json. 👀 [W513] "gulpfile.js" found in repo! Think about migrating to @iobroker/adapter-dev package

TESTING should be updated to current standards. If no technical reason exists, please use standard test-and-release workflow baed on @iobroker/testing 4.x.x.

Alt least testing MUST be adapterd to be performed on all suppoerted platforms 8Linux, window, Mac). Currently tests are executed at Linux only.

@Apollon77 Please check whats possible. If it makes sense to provide a PR based on standard tests please let me know