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

homebridge-appletv-enhanced #666

Closed
maxileith opened this issue Dec 29, 2023 · 19 comments
Closed

homebridge-appletv-enhanced #666

maxileith opened this issue Dec 29, 2023 · 19 comments
Labels
awaiting-changes use after review has started - awaiting user to make changes to plugin

Comments

@maxileith
Copy link

Link To GitHub Repo

https://github.com/maxileith/homebridge-appletv-enhanced

Link To NPM Package

https://www.npmjs.com/package/homebridge-appletv-enhanced

Plugin Icon (Optional)

No response

@maxileith maxileith added the pending the label given to a new verification/icon request label Dec 29, 2023
Copy link

✅ Pre-checks completed successfully.

@bwp91
Copy link
Contributor

bwp91 commented Dec 29, 2023

Hi @maxileith

your readme states the plugin only works on linux and (maybe) mac - ok.

i don't have a windows to test, but what happens if a windows user tries to install this plugin? at what point does it not work?

@maxileith
Copy link
Author

maxileith commented Dec 29, 2023

npm won't allow a windows user to install the plugin. In the package.json is specified that the only allowed operating systems are MacOS and Linux.

https://docs.npmjs.com/cli/v10/configuring-npm/package-json#os

@maxileith
Copy link
Author

maxileith commented Dec 29, 2023

And from the issues raised on GitHub I can tell that actually pretty many users are using MacOS. The issues that were raised by MacOS users in the past were mostly non Mac-specific. So I assume that it runs fine (the raised issues have of course been fixed). But I do not have a Mac to test ...

@bwp91
Copy link
Contributor

bwp91 commented Dec 29, 2023

In fact I had just been looking at that os option in the package json, but didn't know if it meant it refused to install on windows, or just printed a warning much like the engines option

@maxileith
Copy link
Author

maxileith commented Dec 29, 2023

It refuses to install. E.g., when setting the os option to ["win32"] the following will happen when trying to install the package:

❯ npm install -g homebridge-appletv-enhanced-1.0.0.tgz
npm ERR! code EBADPLATFORM
npm ERR! notsup Unsupported platform for [email protected]: wanted {"os":"win32"} (current: {"os":"linux"})
npm ERR! notsup Valid os:  win32
npm ERR! notsup Actual os: linux

npm ERR! A complete log of this run can be found in: /home/maxileith/.npm/_logs/2023-12-29T18_38_13_224Z-debug-0.log

Of course, in the homebridge-appletv-enhanced package it is the other way around ("os": ["linux","darwin"]). The above was for demonstration purposes

@bwp91 bwp91 added currently-reviewing use for starting a review - adds a comment with the verification list with empty checkboxes and removed pending the label given to a new verification/icon request labels Dec 30, 2023
Copy link

github-actions bot commented Dec 30, 2023

  • General
    • The plugin must be of type dynamic platform.
    • The plugin must not offer the same nor less functionality than that of any existing verified plugin.
  • Repo
    • The plugin must be published to NPM and the source code available on a GitHub repository, with issues enabled.
    • A GitHub release should be created for every new version of your plugin, with release notes.
  • Environment
    • The plugin must run on all supported LTS versions of Node.js, at the time of writing this is Node v18 and v20.
    • The plugin must successfully install and not start unless it is configured.
    • The plugin must not execute post-install scripts that modify the users' system in any way.
    • The plugin must not require the user to run Homebridge in a TTY or with non-standard startup parameters, even for initial configuration.
  • Codebase
    • The plugin must implement the Homebridge Plugin Settings GUI.
    • The plugin must not contain any analytics or calls that enable you to track the user.
    • If the plugin needs to write files to disk (cache, keys, etc.), it must store them inside the Homebridge storage directory.
    • The plugin must not throw unhandled exceptions, the plugin must catch and log its own errors.



Comment /check to run checks again.

@bwp91
Copy link
Contributor

bwp91 commented Dec 30, 2023

The plugin must not execute post-install scripts that modify the users' system in any way.

Just checking on this one, your plugin only creates new files in the Homebridge storage directory right? it doesn't make any changes to file outside of this?

@maxileith
Copy link
Author

it doesn't make any changes to file outside of this?

That is correct.

This is why I was raising homebridge/homebridge-apt-pkg#16 in the first place.

@bwp91 bwp91 added verified use when a plugin meets the criteria - adds the verified badge text and removed currently-reviewing use for starting a review - adds a comment with the verification list with empty checkboxes labels Dec 30, 2023
Copy link

Congratulations! Your plugin has been verified.

You can now add one of the Verified by Homebridge badges to your plugin's README:

verified-by-homebridge

[![verified-by-homebridge](https://badgen.net/badge/homebridge/verified/purple)](https://github.com/homebridge/homebridge/wiki/Verified-Plugins)

verified-by-homebridge

[![verified-by-homebridge](https://img.shields.io/badge/homebridge-verified-blueviolet?color=%23491F59&style=for-the-badge&logoColor=%23FFFFFF&logo=homebridge)](https://github.com/homebridge/homebridge/wiki/Verified-Plugins)

Your plugin is now also eligible to display a ❤️ Donate button on its tile in the Homebridge UI. See https://github.com/homebridge/homebridge/wiki/Donation-Links for instructions.

If for any reason in the future you can no longer maintain your plugin, please consider transferring it to our unmaintained plugins repo. We can take ownership until another willing developer comes along.

Don't forget to join the official Homebridge Discord server, where plugin developers can get tips and advice from other developers and the Homebridge project team in the #plugin-development channel!

Thank you for your contribution to the Homebridge Community.
https://homebridge.io

bwp91 added a commit that referenced this issue Dec 30, 2023
@bwp91
Copy link
Contributor

bwp91 commented Dec 30, 2023

Nice idea with the venv for python!

@maxileith
Copy link
Author

/check

@github-actions github-actions bot added pending the label given to a new verification/icon request and removed verified use when a plugin meets the criteria - adds the verified badge text labels Jul 12, 2024
Copy link

🔴 The following pre-checks failed:

  • Package JSON: failed to import plugin as ENOENT: no such file or directory, open '/workspace/test-area/node_modules/https:/github.com/maxileith/homebridge-appletv-enhanced/package.json'
  • NPM Package: could not request information as Unexpected end of JSON input
  • Config Schema JSON: missing file

🟢 The following pre-checks passed:

  • Installation: successfully installed
  • Dependencies: homebridge was not installed as a dependency
  • Dependencies: hap-nodejs was not installed as a dependency

⚠️ Please action these failures and then comment /check to run the checks again. Let us know if you need any help.

If updating your package.json and config.schema.json files, don't forget to publish a new version to NPM.

@github-actions github-actions bot added awaiting-changes use after review has started - awaiting user to make changes to plugin and removed pending the label given to a new verification/icon request labels Jul 12, 2024
@maxileith
Copy link
Author

/check

@github-actions github-actions bot added pending the label given to a new verification/icon request and removed awaiting-changes use after review has started - awaiting user to make changes to plugin labels Jul 31, 2024
Copy link

🔴 The following pre-checks failed:

  • Package JSON: failed to import plugin as ENOENT: no such file or directory, open '/workspace/test-area/node_modules/https:/github.com/maxileith/homebridge-appletv-enhanced/package.json'
  • NPM Package: could not request information as Unexpected end of JSON input
  • Config Schema JSON: missing file

🟢 The following pre-checks passed:

  • Installation: successfully installed
  • Dependencies: homebridge was not installed as a dependency
  • Dependencies: hap-nodejs was not installed as a dependency

⚠️ Please action these failures and then comment /check to run the checks again. Let us know if you need any help.

If updating your package.json and config.schema.json files, don't forget to publish a new version to NPM.

@github-actions github-actions bot added the awaiting-changes use after review has started - awaiting user to make changes to plugin label Jul 31, 2024
@github-actions github-actions bot removed the pending the label given to a new verification/icon request label Jul 31, 2024
@maxileith
Copy link
Author

Hi @bwp91. I just wanted to retrigger the check to see if my plugin is still compliant or if I have to comply to new requirements regarding verification. However, I do not really understand the error above as my repo definitely does have a package.json.

@maxileith
Copy link
Author

/check

@github-actions github-actions bot added pending the label given to a new verification/icon request and removed awaiting-changes use after review has started - awaiting user to make changes to plugin labels Sep 13, 2024
Copy link

🔴 The following checks failed:

  • Package JSON: failed to import plugin as ENOENT: no such file or directory, open '/workspace/test-area/node_modules/https:/github.com/maxileith/homebridge-appletv-enhanced/package.json'
  • NPM Package: could not request information as Unexpected end of JSON input
  • Config Schema JSON: missing file

🟢 The following checks passed:

  • Installation: successfully installed
  • Dependencies: homebridge was not installed as a dependency
  • Dependencies: hap-nodejs was not installed as a dependency

⚠️ Please action these failures and then comment /check to run the checks again. Let us know if you need any help.

If updating your package.json and config.schema.json files, don't forget to publish a new version to NPM.

@github-actions github-actions bot added awaiting-changes use after review has started - awaiting user to make changes to plugin and removed pending the label given to a new verification/icon request labels Sep 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-changes use after review has started - awaiting user to make changes to plugin
Projects
None yet
Development

No branches or pull requests

2 participants