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 #120

Closed
7 tasks done
ioBroker-Bot opened this issue Sep 12, 2024 · 7 comments
Closed
7 tasks done

Please consider fixing issues detected by repository checker #120

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

Comments

@ioBroker-Bot
Copy link

ioBroker-Bot commented Sep 12, 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 wireguard 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.wireguard

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

ERRORS:

  • ❗ [E050] '@iobroker/plugin-sentry' must not be listed as dependency at package.json as it will crash js-controller 7 systems. Please remove and create new release.
  • ❗ [E157] js-controller 3.3.22 listed as dependency but 4.0.24 is required as minimum, 5.0.19 is recommended. Please update dependency at io-package.json.
  • ❗ [E530] .releaseconfig.json is no valid json file - SyntaxError: Expected property name or '}' in JSON at position 3.

WARNINGS:

  • 👀 [W034] @iobroker/adapter-core 3.1.4 specified. 3.1.6 is recommended. Please consider updating dependencies at package.json
  • 👀 [W040] "keywords" within package.json should contain "ioBroker"
  • 👀 [W132] Many "common.news" found in io-package.json. Repository builder will truncate at 7 news. Please remove old news.
  • 👀 [W951] .npmignore found but "files" is used at package.json. Please remove .npmignore.

👍 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 Tue, 01 Oct 2024 11:13:11 GMT based on commit 7c62530
ioBroker.repochecker 3.1.0

@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
[E050] '@iobroker/plugin-sentry' must not be listed as dependency at package.json as it will crash js-controller 7 systems. Please remove and create new release.

Grizzelbee added a commit that referenced this issue Oct 1, 2024
Grizzelbee added a commit that referenced this issue Oct 1, 2024
Grizzelbee added a commit that referenced this issue Oct 1, 2024
* (grizzelbee) Upd: Dependencies got updated
* (grizzelbee) Fix: [#120](#120) Fixed some issues mentioned by adapter-checker
@ioBroker-Bot
Copy link
Author

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

@Grizzelbee
Copy link
Owner

@ioBroker-Bot recheck

@Grizzelbee Grizzelbee reopened this Oct 1, 2024
@ioBroker-Bot
Copy link
Author

This issue has been updated by ioBroker Check and Service Bot

The following issues have been fixed
[E050] '@iobroker/plugin-sentry' must not be listed as dependency at package.json as it will crash js-controller 7 systems. Please remove and create new release.
[E157] js-controller 3.3.22 listed as dependency but 4.0.24 is required as minimum, 5.0.19 is recommended. Please update dependency at io-package.json.
[E530] .releaseconfig.json is no valid json file - SyntaxError: Expected property name or '}' in JSON at position 3.
[W034] @iobroker/adapter-core 3.1.4 specified. 3.1.6 is recommended. Please consider updating dependencies at package.json
[W132] Many "common.news" found in io-package.json. Repository builder will truncate at 7 news. Please remove old news.

👍Thanks for fixing the issues.

RECHECK has been performed as requested.

@Grizzelbee
Copy link
Owner

@ioBroker-Bot recheck

@ioBroker-Bot
Copy link
Author

This issue has been updated by ioBroker Check and Service Bot

The following issues have been fixed
[W040] "keywords" within package.json should contain "ioBroker"
[W951] .npmignore found but "files" is used at package.json. Please remove .npmignore.

👍Thanks for fixing the issues.

RECHECK has been performed as requested.

@ioBroker-Bot
Copy link
Author

All issues reported earlier seem to be fixed now.
THANKS for your support.
This issue will be closed.

your
ioBroker Check and Service Bot

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