Please consider fixing issues detected by repository checker
Closed this issue · 5 comments
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 broadlink2 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.broadlink2
ERRORS:
- ❗ [E184] "common.subscribe" will be removed with js-controller >= 6. Please remove from io-package.json and adapt code if required.
- ❗ [E254] Version "2.0.1" listed at common.news at io-package.json does not exist at NPM. Please remove from news section.
WARNINGS:
- 👀 [W113] Adapter should support compact mode
- 👀 [W132] Many "common.news" found in io-package.json. Repository builder will truncate at 7 news. Please remove old news.
- 👀 [W184] "common.materialize" is deprecated for admin >= 5 at io-package.json. Please use property "adminUI".
- 👀 [W504] setInterval found in "broadlink2.js", but no clearInterval detected
- 👀 [W521] "gulp" found at devDependencies at package.json but no "gulpfile.js" found. Is this dependency really required?
- 👀 [W953] .npmignore found - consider using package.json object "files" instead.
👍 No suggestions found
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.
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 Fri, 04 Oct 2024 02:18:11 GMT based on commit a41b006
ioBroker.repochecker 3.1.0
This issue has been updated by ioBroker Check and Service Bot
The following issues have been fixed
[E955] iob_npm.done not found in .npmignore
👍Thanks for fixing the issues.
This issue has been updated by ioBroker Check and Service Bot
The following issues have been fixed
[E184] "common.subscribe" will be removed with js-controller >= 6. Please remove from io-package.json and adapt code if required.
[E254] Version "2.0.1" listed at common.news at io-package.json does not exist at NPM. Please remove from news section.
[W132] Many "common.news" found in io-package.json. Repository builder will truncate at 7 news. Please remove old news.
[W184] "common.materialize" is deprecated for admin >= 5 at io-package.json. Please use property "adminUI".
👍Thanks for fixing the issues.
@ioBroker-Bot recreate
Thanks for contacting me.
I received your request to recheck this repository and to recreate the checker issue. I will start doing my work soon.
I will close an existing issue and create a new one if any issues are to be reported.
Thanks for spending your time and working at an ioBroker adapter.
your
ioBroker Check and Service Bot
Issue outdated due to RECREATE request. Follow up issue #371 has been created.
This issue will be closed.
your
ioBroker Check and Service Bot