Skip to content
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

Please consider fixing issues detected by repository checker #746

Open
9 of 14 tasks
ioBroker-Bot opened this issue Sep 11, 2024 · 7 comments
Open
9 of 14 tasks

Please consider fixing issues detected by repository checker #746

ioBroker-Bot opened this issue Sep 11, 2024 · 7 comments

Comments

@ioBroker-Bot
Copy link

ioBroker-Bot commented Sep 11, 2024

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 ring 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.ring

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

ERRORS:

WARNINGS:

  • 👀 [W047] dependency should not require a specific version. Use "~1.2.3" or "^1.2.3" syntax. Please update "@homebridge/camera-utils:2.2.6"
  • 👀 [W135] "common.tier" is required in io-package.json. Please check https://github.com/ioBroker/ioBroker.docs/blob/master/docs/en/dev/objectsschema.md#adapter.
  • 👀 [W173] Potential sensitive data "password" not listed at "protectedNative" in io-package.json
  • 👀 [W174] Potential sensitive data "password" not listed at "encryptedNative" in io-package.json
  • 👀 [W181] "common.license" in io-package.json is deprecated. Please define object "common.licenseInformation"
  • 👀 [W184] "common.main" is deprecated and ignored. Please remove from io-package.json. Executable is defined by entry "main" at package.json.
  • 👀 [W184] "common.main" is deprecated and ignored. Please remove from io-package.json. Use "main" at package.json instead.
  • 👀 [W184] "common.materializeTab" is deprecated for admin >= 5 at io-package.json. Please use property "adminUI".
  • 👀 [W513] "gulpfile.js" found in repo! Think about migrating to @iobroker/adapter-dev package
  • 👀 [W529] Plugin "manual-review" missing at .releaseconfig.json. Please add.

SUGGESTIONS:

  • 📌 [S526] Consider adding plugin "@alcalzone/release-script-plugin-manual-review".

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
or
@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, 15 Nov 2024 08:45:52 GMT based on commit 97bd395
ioBroker.repochecker 3.2.2

@ioBroker-Bot
Copy link
Author

This issue has been updated by ioBroker Check and Service Bot

The following issues have been fixed
[W184] "common.main" is deprecated and ignored. Please remove from io-package.json. Use "main" at package.json instead.

👍Thanks for fixing the issues.

The following issues are new and have been added
[W184] "common.main" is deprecated and ignored. Please remove from io-package.json. Executable is defined by entry "main" at package.json.

1 similar comment
@ioBroker-Bot
Copy link
Author

This issue has been updated by ioBroker Check and Service Bot

The following issues have been fixed
[W184] "common.main" is deprecated and ignored. Please remove from io-package.json. Use "main" at package.json instead.

👍Thanks for fixing the issues.

The following issues are new and have been added
[W184] "common.main" is deprecated and ignored. Please remove from io-package.json. Executable is defined by entry "main" at package.json.

theimo1221 added a commit that referenced this issue Oct 31, 2024
theimo1221 added a commit that referenced this issue Oct 31, 2024
* (theimo1221) Update pipelines to stop using node 18.x
* (theimo1221) #746 Fix license information in io-package.json
* (theimo1221) #746 Add tier in io-package.json
* (theimo1221) #746 Remove deprecated property 'materializeTab' in io-package.json
* (theimo1221) #746 Remove deprecated property 'main' in io-package.json
* (theimo1221) #746 Bump minimum js-controller in io-package.json
* (theimo1221) #746 Remove specific version reference to camera utils
* (theimo1221) #746 Add @alcalzone/release-script-plugin-manual-review
@theimo1221
Copy link
Collaborator

@ioBroker-Bot recheck

@ioBroker-Bot
Copy link
Author

This issue has been updated by ioBroker Check and Service Bot

The following issues are new and have been added
[E254] Version "6.2.2" listed at common.news at io-package.json does not exist at NPM. Please remove from news section.
[W529] Plugin "manual-review" missing at .releaseconfig.json. Please add.

RECHECK has been performed as requested.

@ioBroker-Bot
Copy link
Author

This issue has been updated by ioBroker Check and Service Bot

The following issues are not fixed and have been reopened
[E254] Version "6.2.2" listed at common.news at io-package.json does not exist at NPM. Please remove from news section.

@ioBroker-Bot
Copy link
Author

This issue has been updated by ioBroker Check and Service Bot

The following issues have been fixed
[E254] Version "6.2.2" listed at common.news at io-package.json does not exist at NPM. Please remove from news section.

👍Thanks for fixing the issues.

@ioBroker-Bot
Copy link
Author

This issue has been updated by ioBroker Check and Service Bot

The following issues are new and have been added
[E804] Please process issue [#747] "Compatibility check and testing for node.js 22"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants