Excodibur / ioBroker.schwoerer-ventcube

Schwörerhaus VentCube IOBroker Adapter
MIT License
8 stars 6 forks source link

Please consider fixing issues detected by repository checker #393

Closed ioBroker-Bot closed 1 month ago

ioBroker-Bot commented 1 month ago

Notification from ioBroker Check and Service Bot

Dear adapter developer,

I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter schwoerer-ventcube for common errors and appropiate suggestions to keep this adapter up to date.

This check is based the current head revisions (master / main branch) of the adapter repository

Please see the result of the check below.

ioBroker.schwoerer-ventcube

Downloads Number of Installations (latest) Number of Installations (stable) - Test and Release

ERRORS:

WARNINGS:

SUGGESTIONS:

Please review issues reported and consider fixing them as soon as appropiate.

Errors reported by repository checker should be fixed as soon as possible. Some of them require a new release to be considered as fixed. Please note that errors reported by checker might be considered as blocking point for future updates at stable repository.

Warnings reported by repository checker should be reviewed. While some warnings can be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.

Suggestions reported by repository checker should be reviewed. Suggestions can be ignored due to a decision of the developer but they are reported as a hint to use a configuration which might get required in future or at least is used be most adapters. Suggestions are always optional to follow.

You may start a new check or force the creation of a new issue at any time by adding the following comment to this issue:

@iobroker-bot recheck @iobroker-bot recreate

Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.

Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.

And THANKS A LOT for maintaining this adapter from me and all users. Let's work together for the best user experience.

your ioBroker Check and Service Bot

@mcm1957 for evidence

Last update at Sat, 28 Sep 2024 08:35:59 GMT based on commit ae90e82c5bb95668e2bf974062d274ca85ae226b ioBroker.repochecker 3.0.7

Excodibur commented 1 month ago

@iobroker-bot recheck

ioBroker-Bot commented 1 month ago

This issue has been updated by ioBroker Check and Service Bot

The following issues have been fixed [E029] Node.js 16 is required as minimum, node.js 18 is recommended. Please adapt "{'engines' : { 'node' >= '10.0.0' } }" at package.json. [E157] js-controller 3.0.0 listed as dependency but 4.0.24 is required as minimum, 5.0.19 is recommended. Please update dependency at io-package.json. [E525] "@alcalzone/release-script" requires plugin "@alcalzone/release-script-plugin-license". Please add. [W040] "keywords" within package.json should contain "ioBroker" [W127] Missing suggested translation into uk of "common.titleLang" in io-package.json. [W134] Missing suggested translation into uk of "common.desc" in io-package.json.

:thumbsup:Thanks for fixing the issues.

RECHECK has been performed as requested.

Excodibur commented 1 month ago

E606 seems to ignore that the adapter-version is documented in a separate file CHANGELOG.md ---> https://github.com/ioBroker/ioBroker.repochecker/issues/302

Excodibur commented 1 month ago

@ioBroker-Bot recheck

ioBroker-Bot commented 1 month ago

This issue has been updated by ioBroker Check and Service Bot

The following issues have been fixed [E606] Current adapter version 1.4.2 not found in README.md [W181] "common.license" in io-package.json is deprecated. Please define object "common.licenseInformation" [W184] "common.materialize" is deprecated for admin >= 5 at io-package.json. Please use property "adminUI". [W521] "gulp" found at devDependencies at package.json but no "gulpfile.js" found. Is this dependency really required? [S522] Please consider migrating to admin 5 UI (jsonConfig). [S526] Consider adding plugin "@alcalzone/release-script-plugin-manual-review".

:thumbsup:Thanks for fixing the issues.

The following issues are new and have been added [E512] "admin/jsonCustom.json5" not found, but custom support is declared

RECHECK has been performed as requested.

Excodibur commented 1 month ago

@ioBroker-Bot recheck

ioBroker-Bot commented 1 month ago

This issue has been updated by ioBroker Check and Service Bot

The following issues have been fixed [E512] "admin/jsonCustom.json5" not found, but custom support is declared

:thumbsup:Thanks for fixing the issues.

RECHECK has been performed as requested.

Excodibur commented 1 month ago

Fixed most important findings

ioBroker-Bot commented 1 month ago

Thanks for processing and closing this issue.

I will recheck for new issues later. So it's no problem if this issue has been closed premature.

You may start a new check at any time by adding the following comment to this issue:

@iobroker-bot recheck

Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.

Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged. Let's work together for the best user experience.
your
ioBroker Check and Service Bot